)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34541,"name":"Manish","email":"manish.laddha@ibm.com","username":"manishladdha"},"change_message_id":"1edd7b13218c087a324669e4e76e949cf3f90e5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5c24866e_0a08adbb","updated":"2022-02-21 06:02:53.000000000","message":"looks good to me","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"0844344caa9955c09f658333e54811d0d0537934","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fcc15a81_553e2789","updated":"2022-02-22 10:11:09.000000000","message":"Hi Chhavi. All the comments are addressed as per our discussion","commit_id":"72bebe6a481495341907582adef9559bb1a91c6b"},{"author":{"_account_id":14806,"name":"Chhavi Agarwal","email":"chhagarw@in.ibm.com","username":"chhagarw"},"change_message_id":"29b011643f0b0ac76b50c284a568ba03b6b88e7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"59382171_69df2571","updated":"2022-02-22 13:15:43.000000000","message":"looks good","commit_id":"ac43d2c11b051682565064ba2c04adc80275807a"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"4e53b814c71debdf9f8416309bf4128a140d4122","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9560395d_756ece22","updated":"2022-04-12 16:55:21.000000000","message":"run-IBM Storage CI","commit_id":"d1e11bac1faea7aecafff2d868450e5a75946d71"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"edba5660e62ebbbe3eb67d545f980687781abba6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"dfa08898_e30ad656","updated":"2022-04-19 10:41:57.000000000","message":"Code changes look good. Zuul and IBM Storage CI have passed. So +1.","commit_id":"785e35537bf80cc4826ecbbfa9386e98adac0a62"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"3e6a4c418e1ea886b9cf960900bd55101b38ffdc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8d001fa8_21d2def3","updated":"2022-04-19 05:25:34.000000000","message":"Hi Eric, Thankyou for your feedback. I have addressed your review comment and uploaded a new patch. Please let me know if there is anything else needed on this.\n\nThank you.","commit_id":"785e35537bf80cc4826ecbbfa9386e98adac0a62"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"d0305a71cf01e8b6258acfb4e58231ae8f280340","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c7564013_4109b85e","updated":"2022-04-20 15:16:28.000000000","message":"This is in a merge conflict.  It should also have a release note associated with it.","commit_id":"785e35537bf80cc4826ecbbfa9386e98adac0a62"},{"author":{"_account_id":34201,"name":"Mounika Sreeram","email":"sreeram.mounika@ibm.com","username":"sreerammounika"},"change_message_id":"3fd1d5ef2c0ec481d194ac53202073ef1e39eeb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4b315abe_c420148f","updated":"2022-04-20 07:37:43.000000000","message":"Zuul and IBM Storage CI passed. Code changes looks good.","commit_id":"785e35537bf80cc4826ecbbfa9386e98adac0a62"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"a88c109b9521bca4e8a7dc913a7aee9a216c81eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a6faad75_2523dc78","updated":"2022-04-19 08:58:46.000000000","message":"run-IBM Storage CI","commit_id":"785e35537bf80cc4826ecbbfa9386e98adac0a62"},{"author":{"_account_id":34541,"name":"Manish","email":"manish.laddha@ibm.com","username":"manishladdha"},"change_message_id":"797d320d9ae935f1ece0fd71846da631d7f2f6a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7dd3172a_f9c9c7b1","updated":"2022-04-19 14:23:45.000000000","message":"the changes looks good","commit_id":"785e35537bf80cc4826ecbbfa9386e98adac0a62"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"bbfe3f1d42cae4eeabe5ed795ecdfe4962178d9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3d752fd3_2252a285","in_reply_to":"c7564013_4109b85e","updated":"2022-04-21 10:56:51.000000000","message":"Thanks for bringing this to my notice. Also, the merge conflict is now resolved.","commit_id":"785e35537bf80cc4826ecbbfa9386e98adac0a62"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"700f0d580a521e93958c2aa94b106152fd1f6662","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"af988be5_59af18da","updated":"2022-05-02 14:38:56.000000000","message":"Just some minor comments...","commit_id":"76de76646d6ed24f9bb3f2fed00e81746e9804a5"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"44ca4e999727738270e01aa7931cbf9a869d994c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0223b44c_8b9fa974","updated":"2022-05-02 15:20:09.000000000","message":"Since the merge conflict has been resolved I am ok with now merging this.  Please address Sean\u0027s comments in a follow-up patch.","commit_id":"76de76646d6ed24f9bb3f2fed00e81746e9804a5"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"c752835fd250653d1ded48e29ed98d35df9d736e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e1a26435_0930418c","updated":"2022-04-21 19:21:25.000000000","message":"recheck","commit_id":"76de76646d6ed24f9bb3f2fed00e81746e9804a5"},{"author":{"_account_id":34541,"name":"Manish","email":"manish.laddha@ibm.com","username":"manishladdha"},"change_message_id":"05da3e0476536de5134c76d8f4a6a80fa1080d56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b616cd09_bbae9513","updated":"2022-04-22 04:54:19.000000000","message":"run-IBM Storage CI","commit_id":"76de76646d6ed24f9bb3f2fed00e81746e9804a5"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"bf435006f7d51e1f8d0aedd6329e5207997f4da1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6486ba0c_04dd7035","in_reply_to":"0223b44c_8b9fa974","updated":"2022-05-02 15:23:32.000000000","message":"Thanks Jay. I will address Sean\u0027s remarks in a follow-up patch.","commit_id":"76de76646d6ed24f9bb3f2fed00e81746e9804a5"}],"cinder/tests/unit/volume/drivers/ibm/test_storwize_svc.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"db71174d58a005c47c193d5d3bf91cc7de7783d6","unresolved":true,"context_lines":[{"line_number":2152,"context_line":"            name \u003d kwargs[\u0027obj\u0027]"},{"line_number":2153,"context_line":"            for k, v in self._rcrelationship_list.items():"},{"line_number":2154,"context_line":"                if ((six.text_type(v[\u0027name\u0027]) \u003d\u003d name) or"},{"line_number":2155,"context_line":"                        (six.text_type(v[\u0027id\u0027]) \u003d\u003d name)):"},{"line_number":2156,"context_line":"                    self._rc_state_transition(\u0027wait\u0027, v)"},{"line_number":2157,"context_line":""},{"line_number":2158,"context_line":"                    if self._next_cmd_error[\u0027lsrcrelationship\u0027] \u003d\u003d \u0027speed_up\u0027:"}],"source_content_type":"text/x-python","patch_set":5,"id":"e8f6c0cb_87bab257","line":2155,"range":{"start_line":2155,"start_character":25,"end_line":2155,"end_character":38},"updated":"2022-04-18 16:10:46.000000000","message":"Since you are changing/adding new code here, maybe it would be better to just use str() instead of six.text_type().","commit_id":"d1e11bac1faea7aecafff2d868450e5a75946d71"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"3e6a4c418e1ea886b9cf960900bd55101b38ffdc","unresolved":false,"context_lines":[{"line_number":2152,"context_line":"            name \u003d kwargs[\u0027obj\u0027]"},{"line_number":2153,"context_line":"            for k, v in self._rcrelationship_list.items():"},{"line_number":2154,"context_line":"                if ((six.text_type(v[\u0027name\u0027]) \u003d\u003d name) or"},{"line_number":2155,"context_line":"                        (six.text_type(v[\u0027id\u0027]) \u003d\u003d name)):"},{"line_number":2156,"context_line":"                    self._rc_state_transition(\u0027wait\u0027, v)"},{"line_number":2157,"context_line":""},{"line_number":2158,"context_line":"                    if self._next_cmd_error[\u0027lsrcrelationship\u0027] \u003d\u003d \u0027speed_up\u0027:"}],"source_content_type":"text/x-python","patch_set":5,"id":"0aa0eb05_f2eb3770","line":2155,"range":{"start_line":2155,"start_character":25,"end_line":2155,"end_character":38},"in_reply_to":"e8f6c0cb_87bab257","updated":"2022-04-19 05:25:34.000000000","message":"Fixed","commit_id":"d1e11bac1faea7aecafff2d868450e5a75946d71"}],"cinder/volume/drivers/ibm/storwize_svc/replication.py":[{"author":{"_account_id":14806,"name":"Chhavi Agarwal","email":"chhagarw@in.ibm.com","username":"chhagarw"},"change_message_id":"bfe1600eacd6b347a5e7ad06142086234644eebf","unresolved":true,"context_lines":[{"line_number":243,"context_line":"            src_change_opts \u003d self.driver._get_vdisk_params("},{"line_number":244,"context_line":"                vref[\u0027volume_type_id\u0027])"},{"line_number":245,"context_line":"            cycle_period_seconds \u003d src_change_opts.get(\u0027cycle_period_seconds\u0027)"},{"line_number":246,"context_line":"            self.driver._helpers.create_relationship("},{"line_number":247,"context_line":"                vref[\u0027name\u0027], target_vol_name, target_system_id,"},{"line_number":248,"context_line":"                self.asyncmirror, True, source_change_vol_name,"},{"line_number":249,"context_line":"                cycle_period_seconds)"}],"source_content_type":"text/x-python","patch_set":2,"id":"0de7f97f_3ff27a61","line":246,"updated":"2022-02-17 09:22:05.000000000","message":"return the rc_id in create_relationship","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"0844344caa9955c09f658333e54811d0d0537934","unresolved":false,"context_lines":[{"line_number":243,"context_line":"            src_change_opts \u003d self.driver._get_vdisk_params("},{"line_number":244,"context_line":"                vref[\u0027volume_type_id\u0027])"},{"line_number":245,"context_line":"            cycle_period_seconds \u003d src_change_opts.get(\u0027cycle_period_seconds\u0027)"},{"line_number":246,"context_line":"            self.driver._helpers.create_relationship("},{"line_number":247,"context_line":"                vref[\u0027name\u0027], target_vol_name, target_system_id,"},{"line_number":248,"context_line":"                self.asyncmirror, True, source_change_vol_name,"},{"line_number":249,"context_line":"                cycle_period_seconds)"}],"source_content_type":"text/x-python","patch_set":2,"id":"e3a29278_6a328047","line":246,"in_reply_to":"0de7f97f_3ff27a61","updated":"2022-02-22 10:11:09.000000000","message":"This is addressed.","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":14806,"name":"Chhavi Agarwal","email":"chhagarw@in.ibm.com","username":"chhagarw"},"change_message_id":"bfe1600eacd6b347a5e7ad06142086234644eebf","unresolved":true,"context_lines":[{"line_number":248,"context_line":"                self.asyncmirror, True, source_change_vol_name,"},{"line_number":249,"context_line":"                cycle_period_seconds)"},{"line_number":250,"context_line":"            # Set target change volume"},{"line_number":251,"context_line":"            vol_attrs \u003d self.driver._helpers.get_vdisk_attributes(vref[\u0027name\u0027])"},{"line_number":252,"context_line":"            self.target_helpers.change_relationship_changevolume("},{"line_number":253,"context_line":"                target_vol_name, target_change_vol_name, False,"},{"line_number":254,"context_line":"                vol_attrs[\u0027RC_name\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"6a971ff4_adac6831","line":251,"updated":"2022-02-17 09:22:05.000000000","message":"Then we will not need this","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"0844344caa9955c09f658333e54811d0d0537934","unresolved":false,"context_lines":[{"line_number":248,"context_line":"                self.asyncmirror, True, source_change_vol_name,"},{"line_number":249,"context_line":"                cycle_period_seconds)"},{"line_number":250,"context_line":"            # Set target change volume"},{"line_number":251,"context_line":"            vol_attrs \u003d self.driver._helpers.get_vdisk_attributes(vref[\u0027name\u0027])"},{"line_number":252,"context_line":"            self.target_helpers.change_relationship_changevolume("},{"line_number":253,"context_line":"                target_vol_name, target_change_vol_name, False,"},{"line_number":254,"context_line":"                vol_attrs[\u0027RC_name\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"19be6b97_2c2d1bba","line":251,"in_reply_to":"6a971ff4_adac6831","updated":"2022-02-22 10:11:09.000000000","message":"This call has been removed as we are taking the rc_id from create_realtionship()","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":14806,"name":"Chhavi Agarwal","email":"chhagarw@in.ibm.com","username":"chhagarw"},"change_message_id":"bfe1600eacd6b347a5e7ad06142086234644eebf","unresolved":true,"context_lines":[{"line_number":251,"context_line":"            vol_attrs \u003d self.driver._helpers.get_vdisk_attributes(vref[\u0027name\u0027])"},{"line_number":252,"context_line":"            self.target_helpers.change_relationship_changevolume("},{"line_number":253,"context_line":"                target_vol_name, target_change_vol_name, False,"},{"line_number":254,"context_line":"                vol_attrs[\u0027RC_name\u0027])"},{"line_number":255,"context_line":"            # Start gmcv relationship"},{"line_number":256,"context_line":"            self.driver._helpers.start_relationship(vref[\u0027name\u0027],"},{"line_number":257,"context_line":"                                                    rc_name\u003d"}],"source_content_type":"text/x-python","patch_set":2,"id":"d30cc761_f205ca23","line":254,"updated":"2022-02-17 09:22:05.000000000","message":"use rc_id instead of rc_name","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"0844344caa9955c09f658333e54811d0d0537934","unresolved":false,"context_lines":[{"line_number":251,"context_line":"            vol_attrs \u003d self.driver._helpers.get_vdisk_attributes(vref[\u0027name\u0027])"},{"line_number":252,"context_line":"            self.target_helpers.change_relationship_changevolume("},{"line_number":253,"context_line":"                target_vol_name, target_change_vol_name, False,"},{"line_number":254,"context_line":"                vol_attrs[\u0027RC_name\u0027])"},{"line_number":255,"context_line":"            # Start gmcv relationship"},{"line_number":256,"context_line":"            self.driver._helpers.start_relationship(vref[\u0027name\u0027],"},{"line_number":257,"context_line":"                                                    rc_name\u003d"}],"source_content_type":"text/x-python","patch_set":2,"id":"2562a3ca_063d9635","line":254,"in_reply_to":"d30cc761_f205ca23","updated":"2022-02-22 10:11:09.000000000","message":"Here we cannot pass the rc_id that we received from create_realtionship() because create_relationship returns the rc_id for the relationship from the primary SVC.\n\nThe RC_name is same on both the primary and target SVC but the rc_id is different in both SVCs. Since here we need the RC_name/rc_id for the target SVC, we are not passing rc_id here which will call the lsvdisk call in change_relationship_changevolume().","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":14806,"name":"Chhavi Agarwal","email":"chhagarw@in.ibm.com","username":"chhagarw"},"change_message_id":"bfe1600eacd6b347a5e7ad06142086234644eebf","unresolved":true,"context_lines":[{"line_number":255,"context_line":"            # Start gmcv relationship"},{"line_number":256,"context_line":"            self.driver._helpers.start_relationship(vref[\u0027name\u0027],"},{"line_number":257,"context_line":"                                                    rc_name\u003d"},{"line_number":258,"context_line":"                                                    vol_attrs[\u0027RC_name\u0027])"},{"line_number":259,"context_line":"        except Exception as e:"},{"line_number":260,"context_line":"            msg \u003d (_(\"Unable to set up gmcv mode replication for %(vol)s. \""},{"line_number":261,"context_line":"                     \"Exception: %(err)s.\") % {\u0027vol\u0027: vref[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"3c0f138c_3dc8ba1b","line":258,"updated":"2022-02-17 09:22:05.000000000","message":"start_relationship(vref[\u0027name\u0027], rc_id)","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"0844344caa9955c09f658333e54811d0d0537934","unresolved":false,"context_lines":[{"line_number":255,"context_line":"            # Start gmcv relationship"},{"line_number":256,"context_line":"            self.driver._helpers.start_relationship(vref[\u0027name\u0027],"},{"line_number":257,"context_line":"                                                    rc_name\u003d"},{"line_number":258,"context_line":"                                                    vol_attrs[\u0027RC_name\u0027])"},{"line_number":259,"context_line":"        except Exception as e:"},{"line_number":260,"context_line":"            msg \u003d (_(\"Unable to set up gmcv mode replication for %(vol)s. \""},{"line_number":261,"context_line":"                     \"Exception: %(err)s.\") % {\u0027vol\u0027: vref[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"7bdf1915_f3cd3acf","line":258,"in_reply_to":"3c0f138c_3dc8ba1b","updated":"2022-02-22 10:11:09.000000000","message":"Here we have passed the rc_id because the call is made on primary SVC. So here rc_id that we got from create_realtionship() works as it is of primary SVC.","commit_id":"513f288ee35af99a5482e61ec3a60279dcd2a7be"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"6e8a04ead803ca28d29ab563d21205b4b143c87c","unresolved":false,"context_lines":[{"line_number":244,"context_line":"                vref[\u0027volume_type_id\u0027])"},{"line_number":245,"context_line":"            cycle_period_seconds \u003d src_change_opts.get(\u0027cycle_period_seconds\u0027)"},{"line_number":246,"context_line":"            rc_name \u003d self.driver._helpers.create_relationship("},{"line_number":247,"context_line":"                    vref[\u0027name\u0027], target_vol_name, target_system_id,"},{"line_number":248,"context_line":"                    self.asyncmirror, True, source_change_vol_name,"},{"line_number":249,"context_line":"                    cycle_period_seconds)"},{"line_number":250,"context_line":"            # Set target change volume"}],"source_content_type":"text/x-python","patch_set":4,"id":"3816103a_c9ce3af3","line":247,"in_reply_to":"0a5c3180_8935b2c9","updated":"2022-02-23 16:05:12.000000000","message":"Thanks Sofia. It has been fixed.","commit_id":"ac43d2c11b051682565064ba2c04adc80275807a"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"626d5dcf563dc82c586afef268b22300d40bbd60","unresolved":true,"context_lines":[{"line_number":244,"context_line":"                vref[\u0027volume_type_id\u0027])"},{"line_number":245,"context_line":"            cycle_period_seconds \u003d src_change_opts.get(\u0027cycle_period_seconds\u0027)"},{"line_number":246,"context_line":"            rc_name \u003d self.driver._helpers.create_relationship("},{"line_number":247,"context_line":"                    vref[\u0027name\u0027], target_vol_name, target_system_id,"},{"line_number":248,"context_line":"                    self.asyncmirror, True, source_change_vol_name,"},{"line_number":249,"context_line":"                    cycle_period_seconds)"},{"line_number":250,"context_line":"            # Set target change volume"}],"source_content_type":"text/x-python","patch_set":4,"id":"0a5c3180_8935b2c9","line":247,"in_reply_to":"12f69815_3ae1ff10","updated":"2022-02-22 19:01:37.000000000","message":"\u003e pep8: E126 continuation line over-indented for hanging indent\n\nPlease fix.","commit_id":"ac43d2c11b051682565064ba2c04adc80275807a"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"700f0d580a521e93958c2aa94b106152fd1f6662","unresolved":true,"context_lines":[{"line_number":191,"context_line":"            except exception.VolumeBackendAPIException as excp:"},{"line_number":192,"context_line":"                if \"CMMVC6035E\" in excp.msg:"},{"line_number":193,"context_line":"                    msg \u003d (\u0027Source change volume: %s already exists\u0027"},{"line_number":194,"context_line":"                           % source_change_vol_name)"},{"line_number":195,"context_line":"                    LOG.info(msg)"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"            # Target volume creation"}],"source_content_type":"text/x-python","patch_set":7,"id":"586e38cd_006855b4","line":194,"range":{"start_line":194,"start_character":27,"end_line":194,"end_character":29},"updated":"2022-05-02 14:38:56.000000000","message":"String formatting should not be performed ahead of time, and there is no need for this msg variable. Would be better to move this into the LOG.info call, and switch the \u0027%\u0027 to a \u0027,\u0027 so the formatting arguments are passed in as an argument. Then the logger can handle actually formatting the string if/when needed.","commit_id":"76de76646d6ed24f9bb3f2fed00e81746e9804a5"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"bf435006f7d51e1f8d0aedd6329e5207997f4da1","unresolved":false,"context_lines":[{"line_number":191,"context_line":"            except exception.VolumeBackendAPIException as excp:"},{"line_number":192,"context_line":"                if \"CMMVC6035E\" in excp.msg:"},{"line_number":193,"context_line":"                    msg \u003d (\u0027Source change volume: %s already exists\u0027"},{"line_number":194,"context_line":"                           % source_change_vol_name)"},{"line_number":195,"context_line":"                    LOG.info(msg)"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"            # Target volume creation"}],"source_content_type":"text/x-python","patch_set":7,"id":"45bed501_a3e9fdca","line":194,"range":{"start_line":194,"start_character":27,"end_line":194,"end_character":29},"in_reply_to":"586e38cd_006855b4","updated":"2022-05-02 15:23:32.000000000","message":"Thanks Sean. I will address this in a follow-up patch.","commit_id":"76de76646d6ed24f9bb3f2fed00e81746e9804a5"}]}