)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32036,"name":"katari manoj kumar","email":"katkumar@in.ibm.com","username":"katarimanojkumar"},"change_message_id":"7e796e993cb845a03238e4dfe8d364a9598b94e1","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[SVF]: Fixed host and group failover issues"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"[Spectrum Virtualize Family] Fixed issues regarding group-failover"},{"line_number":10,"context_line":"and host-failover that impacts IBM SVC storage backend performance"},{"line_number":11,"context_line":"and applies to both GM and GMCV volume replication types."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"closes bug: #1898746"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_35888269","line":10,"updated":"2020-10-09 15:00:36.000000000","message":"\u0027IBM SVC\u0027 can be removed","commit_id":"b7c117bf0ec5848ad7f14a1da83b83d581bd918c"}],"cinder/tests/unit/volume/drivers/ibm/test_storwize_svc.py":[{"author":{"_account_id":32171,"name":"Girish Chilukuri","email":"girish.chilukuri@ibm.com","username":"GirishChilukuri"},"change_message_id":"9078c8801d4378d887ef411b84e093c5429c3761","unresolved":false,"context_lines":[{"line_number":10448,"context_line":"                       \u0027start_relationship\u0027)"},{"line_number":10449,"context_line":"    @mock.patch.object(storwize_svc_common.StorwizeHelpers,"},{"line_number":10450,"context_line":"                       \u0027stop_relationship\u0027)"},{"line_number":10451,"context_line":"    @mock.patch.object(storwize_svc_common.StorwizeHelpers,"},{"line_number":10452,"context_line":"                       \u0027get_relationship_info\u0027)"},{"line_number":10453,"context_line":"    def test_failover_host_by_force_access(self, get_relationship_info,"},{"line_number":10454,"context_line":"                                           stop_relationship,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f6b1bfe_a73fe2d4","line":10451,"updated":"2020-10-14 10:10:25.000000000","message":"Hi Krishna, \n\"get_relationship_info\" function is not used in test case, this mock can be removed.","commit_id":"ac9a28389496753b0b64dadc501f38d976981b6e"},{"author":{"_account_id":32171,"name":"Girish Chilukuri","email":"girish.chilukuri@ibm.com","username":"GirishChilukuri"},"change_message_id":"9078c8801d4378d887ef411b84e093c5429c3761","unresolved":false,"context_lines":[{"line_number":10459,"context_line":"        context \u003d mock.Mock"},{"line_number":10460,"context_line":"        replica_obj.failover_volume_host(context, mm_vol)"},{"line_number":10461,"context_line":"        stop_relationship.assert_called_once_with(target_vol, access\u003dTrue)"},{"line_number":10462,"context_line":"        self.assertTrue(start_relationship.called)"},{"line_number":10463,"context_line":""},{"line_number":10464,"context_line":"    @mock.patch.object(storwize_svc_common.StorwizeSVCCommonDriver,"},{"line_number":10465,"context_line":"                       \u0027_update_volume_stats\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f6b1bfe_e720bab3","line":10462,"range":{"start_line":10462,"start_character":8,"end_line":10462,"end_character":50},"updated":"2020-10-14 10:10:25.000000000","message":"This statement can be replaced with \"start_relationship.assert_called()\"","commit_id":"ac9a28389496753b0b64dadc501f38d976981b6e"}],"cinder/volume/drivers/ibm/storwize_svc/replication.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"30cd936ab18da3b352f3f6f8a8327ade44b94089","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def failover_volume_host(self, context, vref):"},{"line_number":40,"context_line":"        LOG.debug(\u0027enter: failover_volume_host: vref\u003d%(vref)s\u0027,"},{"line_number":41,"context_line":"                  {\u0027vref\u0027: vref[\u0027name\u0027]})"},{"line_number":42,"context_line":"        # Make the aux volume writeable."},{"line_number":43,"context_line":"        try:"},{"line_number":44,"context_line":"            tgt_volume \u003d storwize_const.REPLICA_AUX_VOL_PREFIX + vref.name"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_af90d7ee","line":41,"updated":"2020-11-09 16:33:09.000000000","message":"you could use the cinder.utils.trace decorator for this.\nIt debug logs entry and exit of functions.","commit_id":"00896d52ed2bb98d578d59bc0a4257a43edd9008"},{"author":{"_account_id":32266,"name":"Venkata krishna Thumu","display_name":"VenkataKrishna","email":"venkata.krishna.reddy@ibm.com","username":"venkatakrishnathumu","status":"Active"},"change_message_id":"d637cdabc7bc7276aa049839fe19431c989bad90","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def failover_volume_host(self, context, vref):"},{"line_number":40,"context_line":"        LOG.debug(\u0027enter: failover_volume_host: vref\u003d%(vref)s\u0027,"},{"line_number":41,"context_line":"                  {\u0027vref\u0027: vref[\u0027name\u0027]})"},{"line_number":42,"context_line":"        # Make the aux volume writeable."},{"line_number":43,"context_line":"        try:"},{"line_number":44,"context_line":"            tgt_volume \u003d storwize_const.REPLICA_AUX_VOL_PREFIX + vref.name"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_9caf057c","line":41,"in_reply_to":"1f621f24_af90d7ee","updated":"2020-11-20 05:58:54.000000000","message":"Updated","commit_id":"00896d52ed2bb98d578d59bc0a4257a43edd9008"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"30cd936ab18da3b352f3f6f8a8327ade44b94089","unresolved":false,"context_lines":[{"line_number":46,"context_line":"            try:"},{"line_number":47,"context_line":"                self.target_helpers.start_relationship(tgt_volume, \u0027aux\u0027)"},{"line_number":48,"context_line":"            except exception.VolumeBackendAPIException as e:"},{"line_number":49,"context_line":"                LOG.error(\u0027Error running startrcrelationship due to %(err)s.\u0027,"},{"line_number":50,"context_line":"                          {\u0027err\u0027: e})"},{"line_number":51,"context_line":"            return"},{"line_number":52,"context_line":"        except Exception as e:"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_efbf6f5e","line":49,"updated":"2020-11-09 16:33:09.000000000","message":"startrcrelationship ?\nwhy not start create relationship?","commit_id":"00896d52ed2bb98d578d59bc0a4257a43edd9008"},{"author":{"_account_id":32266,"name":"Venkata krishna Thumu","display_name":"VenkataKrishna","email":"venkata.krishna.reddy@ibm.com","username":"venkatakrishnathumu","status":"Active"},"change_message_id":"d637cdabc7bc7276aa049839fe19431c989bad90","unresolved":false,"context_lines":[{"line_number":46,"context_line":"            try:"},{"line_number":47,"context_line":"                self.target_helpers.start_relationship(tgt_volume, \u0027aux\u0027)"},{"line_number":48,"context_line":"            except exception.VolumeBackendAPIException as e:"},{"line_number":49,"context_line":"                LOG.error(\u0027Error running startrcrelationship due to %(err)s.\u0027,"},{"line_number":50,"context_line":"                          {\u0027err\u0027: e})"},{"line_number":51,"context_line":"            return"},{"line_number":52,"context_line":"        except Exception as e:"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f621f24_5c63ad42","line":49,"in_reply_to":"1f621f24_efbf6f5e","updated":"2020-11-20 05:58:54.000000000","message":"startrcrelationship is the actual IBM SVF backend command running during this process. So that error message is useful while debugging.","commit_id":"00896d52ed2bb98d578d59bc0a4257a43edd9008"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"17bdcf2118a9eb98cfe9fe5501b714d718524831","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    @utils.trace"},{"line_number":40,"context_line":"    def failover_volume_host(self, context, vref):"},{"line_number":41,"context_line":"        LOG.debug(\u0027enter: failover_volume_host: vref\u003d%(vref)s\u0027,"},{"line_number":42,"context_line":"                  {\u0027vref\u0027: vref[\u0027name\u0027]})"},{"line_number":43,"context_line":"        # Make the aux volume writeable."},{"line_number":44,"context_line":"        try:"},{"line_number":45,"context_line":"            tgt_volume \u003d storwize_const.REPLICA_AUX_VOL_PREFIX + vref.name"}],"source_content_type":"text/x-python","patch_set":5,"id":"fffc6b78_20e1089c","line":42,"updated":"2020-11-19 18:22:31.000000000","message":"you can remove this debug log when you use @utils.trace","commit_id":"ad1dae081884f1f67c7506d828fa9f1801575db8"},{"author":{"_account_id":32266,"name":"Venkata krishna Thumu","display_name":"VenkataKrishna","email":"venkata.krishna.reddy@ibm.com","username":"venkatakrishnathumu","status":"Active"},"change_message_id":"d637cdabc7bc7276aa049839fe19431c989bad90","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    @utils.trace"},{"line_number":40,"context_line":"    def failover_volume_host(self, context, vref):"},{"line_number":41,"context_line":"        LOG.debug(\u0027enter: failover_volume_host: vref\u003d%(vref)s\u0027,"},{"line_number":42,"context_line":"                  {\u0027vref\u0027: vref[\u0027name\u0027]})"},{"line_number":43,"context_line":"        # Make the aux volume writeable."},{"line_number":44,"context_line":"        try:"},{"line_number":45,"context_line":"            tgt_volume \u003d storwize_const.REPLICA_AUX_VOL_PREFIX + vref.name"}],"source_content_type":"text/x-python","patch_set":5,"id":"fffc6b78_f5c4dc94","line":42,"in_reply_to":"fffc6b78_20e1089c","updated":"2020-11-20 05:58:54.000000000","message":"Removed","commit_id":"ad1dae081884f1f67c7506d828fa9f1801575db8"}]}