)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"3faf8def7f1c045bb8cbd5770b0b70f8caada7ca","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Volume replication support will be added in next VxFlex OS release."},{"line_number":10,"context_line":"Cinder driver for VxFlex OS supports volumes/snapshots with"},{"line_number":11,"context_line":"replication enabled."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: blueprint vxflexos-replication-support"},{"line_number":14,"context_line":"Change-Id: I3f2ec1ddf1867261253190953f7a65fff22c7404"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_6605c778","line":12,"updated":"2020-02-12 17:18:37.000000000","message":"Depends ON?\nLooks like this patch depends on https://review.opendev.org/#/c/705176/","commit_id":"13246959c8207a895f762448bf483d091c73383e"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"3c481c3a8b546c196ce8b826097f8aad9ee2cde7","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Volume replication support will be added in next VxFlex OS release."},{"line_number":10,"context_line":"Cinder driver for VxFlex OS supports volumes/snapshots with"},{"line_number":11,"context_line":"replication enabled."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: blueprint vxflexos-replication-support"},{"line_number":14,"context_line":"Change-Id: I3f2ec1ddf1867261253190953f7a65fff22c7404"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_d96a5c80","line":12,"in_reply_to":"3fa7e38b_6605c778","updated":"2020-02-12 17:49:45.000000000","message":"As this patch is the part of the patches chain I\u0027m not really sure that it should have Depends-On tag in commit message.","commit_id":"13246959c8207a895f762448bf483d091c73383e"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"71e7bcd0cd861623c5b9f0ec882efc2f8f4e4e83","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ivan Pchelintsev \u003cIvan.Pchelintsev@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-02-13 14:41:05 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add replication v2.1 support in VxFlex OS driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Volume replication support will be added in next VxFlex OS release."},{"line_number":10,"context_line":"Cinder driver for VxFlex OS supports volumes/snapshots with"},{"line_number":11,"context_line":"replication enabled."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: blueprint vxflexos-replication-support"},{"line_number":14,"context_line":"Change-Id: I3f2ec1ddf1867261253190953f7a65fff22c7404"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_e8ca8f79","line":11,"range":{"start_line":7,"start_character":0,"end_line":11,"end_character":20},"updated":"2020-02-18 18:49:13.000000000","message":"is it really replication version 2.1 as the initial version.\nSuggest adding that replication is added starting from vxflexos 3.5 version.","commit_id":"eeadb174980df63ab96f05b6c423842124ff510b"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"a7e7998f9ed8efee5c127626723f3a64df4e5197","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ivan Pchelintsev \u003cIvan.Pchelintsev@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-02-13 14:41:05 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add replication v2.1 support in VxFlex OS driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Volume replication support will be added in next VxFlex OS release."},{"line_number":10,"context_line":"Cinder driver for VxFlex OS supports volumes/snapshots with"},{"line_number":11,"context_line":"replication enabled."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: blueprint vxflexos-replication-support"},{"line_number":14,"context_line":"Change-Id: I3f2ec1ddf1867261253190953f7a65fff22c7404"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_92af7d2c","line":11,"range":{"start_line":7,"start_character":0,"end_line":11,"end_character":20},"in_reply_to":"3fa7e38b_e8ca8f79","updated":"2020-02-19 07:58:25.000000000","message":"OpenStack replication version is currently 2.1 according to specification. It supports only one case when primary backend died and we are going to use secondary (DR). Therefore I have added replication version in commit message and driver code.\nWith next patchset I\u0027ll try to make messages and release notes more clear for end users.","commit_id":"eeadb174980df63ab96f05b6c423842124ff510b"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"9d4d0e51613831b4324a2b37552b73a79153d047","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add OpenStack volume replication v2.1 support in VxFlex OS driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Volume replication support will be added in next VxFlex OS (v3.5.0) release."},{"line_number":10,"context_line":"Cinder driver for VxFlex OS supports volumes/snapshots with"},{"line_number":11,"context_line":"replication enabled according to OpenStack volume replication specification."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: blueprint vxflexos-replication-support"},{"line_number":14,"context_line":"Change-Id: I3f2ec1ddf1867261253190953f7a65fff22c7404"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1fa4df85_a93267a6","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":76},"updated":"2020-02-24 20:34:14.000000000","message":"NIT.\ntext should fit into 80 chat lines.","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"f3cbdc2a6a130424625473fe26f11587fb2ee392","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add OpenStack volume replication v2.1 support in VxFlex OS driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Volume replication support will be added in next VxFlex OS (v3.5.0) release."},{"line_number":10,"context_line":"Cinder driver for VxFlex OS supports volumes/snapshots with"},{"line_number":11,"context_line":"replication enabled according to OpenStack volume replication specification."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Implements: blueprint vxflexos-replication-support"},{"line_number":14,"context_line":"Change-Id: I3f2ec1ddf1867261253190953f7a65fff22c7404"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1fa4df85_92878bd8","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":76},"in_reply_to":"1fa4df85_a93267a6","updated":"2020-02-25 09:15:45.000000000","message":"Fixed","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"}],"cinder/tests/unit/volume/drivers/dell_emc/vxflexos/test_replication.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"8ce441a2629f506775c2c70a813b943879054343","unresolved":false,"context_lines":[{"line_number":100,"context_line":"    def test_failover_host_failback(self):"},{"line_number":101,"context_line":"        self.test_do_setup_already_failed_over()"},{"line_number":102,"context_line":"        self.driver.failover_host({}, [], \u0027default\u0027)"},{"line_number":103,"context_line":"        self.assertEqual(self.driver.active_backend_id, \u0027default\u0027)"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"    @ddt.data(\"not_valid_target\", None)"},{"line_number":106,"context_line":"    def test_failover_host_secondary_id_invalid(self, secondary_id):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_153b92d1","line":103,"range":{"start_line":103,"start_character":25,"end_line":103,"end_character":65},"updated":"2020-02-24 16:15:05.000000000","message":"assertEqual order should be (expected_value, actual_value)","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"476f40fd42eb6c957196e954fd0e2c0eef2e8c52","unresolved":false,"context_lines":[{"line_number":100,"context_line":"    def test_failover_host_failback(self):"},{"line_number":101,"context_line":"        self.test_do_setup_already_failed_over()"},{"line_number":102,"context_line":"        self.driver.failover_host({}, [], \u0027default\u0027)"},{"line_number":103,"context_line":"        self.assertEqual(self.driver.active_backend_id, \u0027default\u0027)"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"    @ddt.data(\"not_valid_target\", None)"},{"line_number":106,"context_line":"    def test_failover_host_secondary_id_invalid(self, secondary_id):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_92eeab86","line":103,"range":{"start_line":103,"start_character":25,"end_line":103,"end_character":65},"in_reply_to":"1fa4df85_153b92d1","updated":"2020-02-25 09:23:47.000000000","message":"Fixed","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"28851501c2337c5bcce8a0eeabb15832f5e188cc","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        }"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_do_setup_replication_configured(self):"},{"line_number":58,"context_line":"        super(vxflexos.mocks.VxFlexOSDriver, self.driver).do_setup({})"},{"line_number":59,"context_line":"        self.driver.check_for_setup_error()"},{"line_number":60,"context_line":"        self.assertTrue(self.driver.secondary_client.is_configured)"},{"line_number":61,"context_line":"        self.assertTrue(self.driver.replication_enabled)"}],"source_content_type":"text/x-python","patch_set":9,"id":"df33271e_0b2b7c82","line":58,"updated":"2020-04-01 13:55:25.000000000","message":"E1003: \u001b[1;31mBad first argument \u0027VxFlexOSDriver\u0027 given to super()\u001b[0m (\u001b[1;31mbad-super-call\u001b[0m)","commit_id":"b872c73d69d284ede156c7c60f6729ee9dd0013c"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"28851501c2337c5bcce8a0eeabb15832f5e188cc","unresolved":false,"context_lines":[{"line_number":87,"context_line":""},{"line_number":88,"context_line":"    def test_do_setup_already_failed_over(self):"},{"line_number":89,"context_line":"        self.driver.active_backend_id \u003d \u0027vxflex_repl\u0027"},{"line_number":90,"context_line":"        super(vxflexos.mocks.VxFlexOSDriver, self.driver).do_setup({})"},{"line_number":91,"context_line":"        self.driver.check_for_setup_error()"},{"line_number":92,"context_line":"        self.assertFalse(self.driver.replication_enabled)"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"df33271e_eb2d7895","line":90,"updated":"2020-04-01 13:55:25.000000000","message":"E1003: \u001b[1;31mBad first argument \u0027VxFlexOSDriver\u0027 given to super()\u001b[0m (\u001b[1;31mbad-super-call\u001b[0m)","commit_id":"b872c73d69d284ede156c7c60f6729ee9dd0013c"}],"cinder/volume/drivers/dell_emc/vxflexos/driver.py":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"71e7bcd0cd861623c5b9f0ec882efc2f8f4e4e83","unresolved":false,"context_lines":[{"line_number":86,"context_line":"          2.0.5 - Change driver name, rename config file options"},{"line_number":87,"context_line":"          3.0.0 - Add support for VxFlex OS 3.0.x and for volumes compression"},{"line_number":88,"context_line":"          3.5.0 - Add support for VxFlex OS 3.5.x"},{"line_number":89,"context_line":"          3.5.1 - Add volume replication v2.1 support"},{"line_number":90,"context_line":"    \"\"\""},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    VERSION \u003d \"3.5.1\""}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_a8a257ab","line":89,"range":{"start_line":89,"start_character":41,"end_line":89,"end_character":46},"updated":"2020-02-18 18:49:13.000000000","message":"is 2.1 the right version?","commit_id":"eeadb174980df63ab96f05b6c423842124ff510b"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"71e7bcd0cd861623c5b9f0ec882efc2f8f4e4e83","unresolved":false,"context_lines":[{"line_number":348,"context_line":"        Create volume on secondary VxFlex OS storage backend."},{"line_number":349,"context_line":"        Pair volumes and add replication pair to replication consistency group."},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        :param protection_domain_name: name of VxFlex OS Protection Domain"},{"line_number":352,"context_line":"        :param storage_pool_name: name of VxFlex OS Storage Pool"},{"line_number":353,"context_line":"        :param volume: volume to be created on storage backend"},{"line_number":354,"context_line":"        :param provisioning: volume provisioning type"},{"line_number":355,"context_line":"        :param compression: volume compression method"},{"line_number":356,"context_line":"        :param source_provider_id: primary VxFlex OS volume id"},{"line_number":357,"context_line":"        :param rcg_name: name of VxFlex OS Replication Consistency Group"},{"line_number":358,"context_line":"        \"\"\""},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"        LOG.info(\"Configure replication for volume %(vol_id)s. \""}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_08036b77","line":357,"range":{"start_line":351,"start_character":0,"end_line":357,"end_character":72},"updated":"2020-02-18 18:49:13.000000000","message":"Any additional requirements, like replicated volume will have the same compression method as original one?\nWhat degrees of freedom between original and new replicated volumes allowed? \nAnd if some combinations are not legal do we rely on backend response to report an error? How does user figures out what parameter combinations are not legal?","commit_id":"eeadb174980df63ab96f05b6c423842124ff510b"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"a7e7998f9ed8efee5c127626723f3a64df4e5197","unresolved":false,"context_lines":[{"line_number":348,"context_line":"        Create volume on secondary VxFlex OS storage backend."},{"line_number":349,"context_line":"        Pair volumes and add replication pair to replication consistency group."},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        :param protection_domain_name: name of VxFlex OS Protection Domain"},{"line_number":352,"context_line":"        :param storage_pool_name: name of VxFlex OS Storage Pool"},{"line_number":353,"context_line":"        :param volume: volume to be created on storage backend"},{"line_number":354,"context_line":"        :param provisioning: volume provisioning type"},{"line_number":355,"context_line":"        :param compression: volume compression method"},{"line_number":356,"context_line":"        :param source_provider_id: primary VxFlex OS volume id"},{"line_number":357,"context_line":"        :param rcg_name: name of VxFlex OS Replication Consistency Group"},{"line_number":358,"context_line":"        \"\"\""},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"        LOG.info(\"Configure replication for volume %(vol_id)s. \""}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_12786dc0","line":357,"range":{"start_line":351,"start_character":0,"end_line":357,"end_character":72},"in_reply_to":"3fa7e38b_08036b77","updated":"2020-02-19 07:58:25.000000000","message":"I will add the documentation later when the driver will be merged.\nFor now both primary and secondary backends must have PDs and SPs configured the same way (same names, zero padding, etc.).\nVolume on destination backend is created with the same provisioning type as original, compression is used only if destination SP supports this feature.","commit_id":"eeadb174980df63ab96f05b6c423842124ff510b"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"9d4d0e51613831b4324a2b37552b73a79153d047","unresolved":false,"context_lines":[{"line_number":120,"context_line":"            properties,"},{"line_number":121,"context_line":"            \"vxflexos:replication_cg\","},{"line_number":122,"context_line":"            \"VxFlex OS Replication Consistency Group.\","},{"line_number":123,"context_line":"            _(\"Specifies the VxFlex OS Replication Consistency group for a \""},{"line_number":124,"context_line":"              \"volume type. Source and target volumes will be added to the \""},{"line_number":125,"context_line":"              \"specified RCG during creation.\"),"},{"line_number":126,"context_line":"            \"string\")"},{"line_number":127,"context_line":"        return properties, \"vxflexos\""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_291e770c","line":126,"range":{"start_line":123,"start_character":0,"end_line":126,"end_character":21},"updated":"2020-02-24 20:34:14.000000000","message":"looks like mismatch on ()\nmaybe _ -\u003e (","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"f3cbdc2a6a130424625473fe26f11587fb2ee392","unresolved":false,"context_lines":[{"line_number":120,"context_line":"            properties,"},{"line_number":121,"context_line":"            \"vxflexos:replication_cg\","},{"line_number":122,"context_line":"            \"VxFlex OS Replication Consistency Group.\","},{"line_number":123,"context_line":"            _(\"Specifies the VxFlex OS Replication Consistency group for a \""},{"line_number":124,"context_line":"              \"volume type. Source and target volumes will be added to the \""},{"line_number":125,"context_line":"              \"specified RCG during creation.\"),"},{"line_number":126,"context_line":"            \"string\")"},{"line_number":127,"context_line":"        return properties, \"vxflexos\""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_d2666352","line":126,"range":{"start_line":123,"start_character":0,"end_line":126,"end_character":21},"in_reply_to":"1fa4df85_291e770c","updated":"2020-02-25 09:15:45.000000000","message":"Does not seems wrong to me.","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"8ce441a2629f506775c2c70a813b943879054343","unresolved":false,"context_lines":[{"line_number":536,"context_line":"                              })"},{"line_number":537,"context_line":"                    snapshot.update({\"status\": fields.SnapshotStatus.ERROR})"},{"line_number":538,"context_line":"                finally:"},{"line_number":539,"context_line":"                    snapshot.save()"},{"line_number":540,"context_line":"        else:"},{"line_number":541,"context_line":"            updates[\"replication_status\"] \u003d error_status"},{"line_number":542,"context_line":"        return updates"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_b59f1eb3","line":539,"range":{"start_line":539,"start_character":20,"end_line":539,"end_character":33},"updated":"2020-02-24 16:15:05.000000000","message":"Do we really need DB access from the driver? Any DB changes should really be passed back to the manager through model updates.","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"476f40fd42eb6c957196e954fd0e2c0eef2e8c52","unresolved":false,"context_lines":[{"line_number":536,"context_line":"                              })"},{"line_number":537,"context_line":"                    snapshot.update({\"status\": fields.SnapshotStatus.ERROR})"},{"line_number":538,"context_line":"                finally:"},{"line_number":539,"context_line":"                    snapshot.save()"},{"line_number":540,"context_line":"        else:"},{"line_number":541,"context_line":"            updates[\"replication_status\"] \u003d error_status"},{"line_number":542,"context_line":"        return updates"}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_726c4f08","line":539,"range":{"start_line":539,"start_character":20,"end_line":539,"end_character":33},"in_reply_to":"1fa4df85_b59f1eb3","updated":"2020-02-25 09:23:47.000000000","message":"I did not find another way of how to pass snapshot updates inside volume updates dict. Maybe I\u0027m missing something.","commit_id":"109866f179c3bed989cd1cecce91c5d780dfe66f"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"56ae67fb69ea9ba6f953b4e9e92ff4340acc20bf","unresolved":false,"context_lines":[{"line_number":537,"context_line":"                              })"},{"line_number":538,"context_line":"                    snapshot.update({\"status\": fields.SnapshotStatus.ERROR})"},{"line_number":539,"context_line":"                finally:"},{"line_number":540,"context_line":"                    snapshot.save()"},{"line_number":541,"context_line":"        else:"},{"line_number":542,"context_line":"            updates[\"replication_status\"] \u003d error_status"},{"line_number":543,"context_line":"        return updates"}],"source_content_type":"text/x-python","patch_set":6,"id":"1fa4df85_5f30cee8","line":540,"updated":"2020-03-03 13:29:42.000000000","message":"If we can\u0027t currently update snapshot information through returning a model update, then that should be fixed. There really should be no DB access from within a driver.","commit_id":"bf1088ef32335c2f7d2a9f5a9e76cd527119074d"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"421939da6c9cb25a26fd7590ab9d3595af1d948d","unresolved":false,"context_lines":[{"line_number":362,"context_line":"                                             secondary\u003dTrue)"},{"line_number":363,"context_line":"            storage_type \u003d self._get_volumetype_extraspecs(entity)"},{"line_number":364,"context_line":"            rcg_name \u003d storage_type.get(REPLICATION_CG_KEY)"},{"line_number":365,"context_line":"            LOG.info(\"Replication Consistency Group name: %s.\", rcg_name)"},{"line_number":366,"context_line":"            provisioning, compression \u003d self._get_provisioning_and_compression("},{"line_number":367,"context_line":"                storage_type,"},{"line_number":368,"context_line":"                protection_domain_name,"}],"source_content_type":"text/x-python","patch_set":10,"id":"df33271e_51f7445a","line":365,"range":{"start_line":365,"start_character":12,"end_line":365,"end_character":20},"updated":"2020-04-08 19:21:16.000000000","message":"This could probably be a LOG.debug","commit_id":"4e320543cf6a115fdaa807ee9df3a8adbbd9e920"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"421939da6c9cb25a26fd7590ab9d3595af1d948d","unresolved":false,"context_lines":[{"line_number":492,"context_line":"        if failover_status \u003d\u003d fields.ReplicationStatus.FAILED_OVER:"},{"line_number":493,"context_line":"            client \u003d self._get_client(secondary\u003dTrue)"},{"line_number":494,"context_line":"            try:"},{"line_number":495,"context_line":"                LOG.info(\"Query new provider_id for volume %s.\", volume.id)"},{"line_number":496,"context_line":"                pair_id, remote_pair_id, vol_id, remote_vol_id \u003d ("},{"line_number":497,"context_line":"                    client.get_volumes_pair_attrs(\"remoteVolumeId\","},{"line_number":498,"context_line":"                                                  volume.provider_id)"}],"source_content_type":"text/x-python","patch_set":10,"id":"df33271e_9168acf5","line":495,"range":{"start_line":495,"start_character":16,"end_line":495,"end_character":24},"updated":"2020-04-08 19:21:16.000000000","message":"This could also be a LOG.debug and the next one as well.","commit_id":"4e320543cf6a115fdaa807ee9df3a8adbbd9e920"}]}