)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"3104dabba95830eceef59e2f2225eff963776b88","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8cf0c0cc_0a901ca1","updated":"2023-07-12 13:32:53.000000000","message":"I think this patch is superseded by https://review.opendev.org/c/openstack/cinder/+/847730\n\nShould this patch be abandoned?","commit_id":"bc30ede9423373cc3336dc30657bcca6c5e14df7"},{"author":{"_account_id":33609,"name":"XuQi","display_name":"Inori","email":"xuq.fnstxz@fujitsu.com","username":"inori"},"change_message_id":"0931087b5cd5041f7769812667323e9d38992c7b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"cc468399_fcb1625b","updated":"2022-07-07 02:36:19.000000000","message":"run-Fujitsu ETERNUS CI","commit_id":"bc30ede9423373cc3336dc30657bcca6c5e14df7"},{"author":{"_account_id":33609,"name":"XuQi","display_name":"Inori","email":"xuq.fnstxz@fujitsu.com","username":"inori"},"change_message_id":"01278ed662ffdf403ec44ae1f51dd7aeea46c9c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e9c5dcac_875e8d07","in_reply_to":"8cf0c0cc_0a901ca1","updated":"2023-07-12 13:53:35.000000000","message":"Yes, this patch is superseded by the patch you mentioned, so it can be abandoned. However, since I am currently unable to access the account of the patch\u0027s owner, I am unable to abandoned it.","commit_id":"bc30ede9423373cc3336dc30657bcca6c5e14df7"}],"cinder/tests/unit/volume/drivers/test_fujitsu_dx.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"d3fb39f581796685b275c7ad9762e0c0e668e2ad","unresolved":false,"context_lines":[{"line_number":1134,"context_line":"    def test_set_qos(self):"},{"line_number":1135,"context_line":"        self.mock_object(dx_common.FJDXCommon, \u0027_get_qos_specs\u0027,"},{"line_number":1136,"context_line":"                         return_value\u003d{\u0027maxBWS\u0027: \u002710000\u0027})"},{"line_number":1137,"context_line":"        self.assertEqual(True, self.driver.common._set_qos("},{"line_number":1138,"context_line":"            TEST_VOLUME, use_id\u003dTrue))"},{"line_number":1139,"context_line":""},{"line_number":1140,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"1f493fa4_bf2e94a2","line":1137,"updated":"2020-04-23 04:20:58.000000000","message":"pep8: C313: Unit tests should use assertTrue(value) instead of using assertEqual(True, value).","commit_id":"f78b6d0031c05f2f0b508cd40041c5c35d2c1fed"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"d3fb39f581796685b275c7ad9762e0c0e668e2ad","unresolved":false,"context_lines":[{"line_number":1370,"context_line":"    def test_set_qos(self):"},{"line_number":1371,"context_line":"        self.mock_object(dx_common.FJDXCommon, \u0027_get_qos_specs\u0027,"},{"line_number":1372,"context_line":"                         return_value\u003d{\u0027maxBWS\u0027: \u002710000\u0027})"},{"line_number":1373,"context_line":"        self.assertEqual(True, self.driver.common._set_qos("},{"line_number":1374,"context_line":"            TEST_VOLUME, use_id\u003dTrue))"}],"source_content_type":"text/x-python","patch_set":7,"id":"1f493fa4_9f2bd892","line":1373,"updated":"2020-04-23 04:20:58.000000000","message":"pep8: C313: Unit tests should use assertTrue(value) instead of using assertEqual(True, value).","commit_id":"f78b6d0031c05f2f0b508cd40041c5c35d2c1fed"}],"cinder/volume/drivers/fujitsu/eternus_dx/eternus_dx_cli.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"b9f53060a7ee290b7fa08b295e77d48584a4f50d","unresolved":false,"context_lines":[{"line_number":304,"context_line":"            output \u003d {\u0027result\u0027: 0,"},{"line_number":305,"context_line":"                      \u0027rc\u0027: \u00274\u0027,"},{"line_number":306,"context_line":"                      \u0027message\u0027: \"Show qos bandwidth limit error: %s\""},{"line_number":307,"context_line":"                                 % six.text_type(ex)}"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"        return output"},{"line_number":310,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1fa4df85_e967bab6","line":307,"updated":"2020-03-17 12:05:31.000000000","message":"six isn\u0027t required now that the current release is python3 only.   This is optional.","commit_id":"945efec4a0d49fd8dca3f14e1dc009e1bf79246a"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"8c16fb858801921ff77d8761108762116984e0e7","unresolved":false,"context_lines":[{"line_number":297,"context_line":""},{"line_number":298,"context_line":"            output[\u0027message\u0027] \u003d qoslist"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"        except Exception as ex:"},{"line_number":301,"context_line":"            output \u003d {\u0027result\u0027: 0,"},{"line_number":302,"context_line":"                      \u0027rc\u0027: \u00274\u0027,"},{"line_number":303,"context_line":"                      \u0027message\u0027: \"Show qos bandwidth limit error: %s\" % ex}"}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_5880738c","line":300,"range":{"start_line":300,"start_character":28,"end_line":300,"end_character":30},"updated":"2020-03-23 08:51:24.000000000","message":"Do we want to add some logging here to log errors?","commit_id":"8c384285a686ce1d97273150adad8f996816100f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"8c16fb858801921ff77d8761108762116984e0e7","unresolved":false,"context_lines":[{"line_number":297,"context_line":""},{"line_number":298,"context_line":"            output[\u0027message\u0027] \u003d qoslist"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"        except Exception as ex:"},{"line_number":301,"context_line":"            output \u003d {\u0027result\u0027: 0,"},{"line_number":302,"context_line":"                      \u0027rc\u0027: \u00274\u0027,"},{"line_number":303,"context_line":"                      \u0027message\u0027: \"Show qos bandwidth limit error: %s\" % ex}"}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_1876ebd9","line":300,"updated":"2020-03-23 08:51:24.000000000","message":"Please, catch more specific exceptions here and below","commit_id":"8c384285a686ce1d97273150adad8f996816100f"}],"cinder/volume/drivers/fujitsu/eternus_dx/eternus_dx_common.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"8c16fb858801921ff77d8761108762116984e0e7","unresolved":false,"context_lines":[{"line_number":101,"context_line":"        d_metadata.update(metadata)"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"        model_update \u003d {"},{"line_number":104,"context_line":"            \u0027provider_location\u0027: six.text_type(element_path),"},{"line_number":105,"context_line":"            \u0027metadata\u0027: d_metadata"},{"line_number":106,"context_line":"        }"},{"line_number":107,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_f85b4751","line":104,"range":{"start_line":104,"start_character":33,"end_line":104,"end_character":36},"updated":"2020-03-23 08:51:24.000000000","message":"You remove six from eternus_dx_cli. I\u0027m OK to have six in drivers but let\u0027s do it consistent","commit_id":"8c384285a686ce1d97273150adad8f996816100f"}],"cinder/volume/drivers/fujitsu/eternus_dx/eternus_dx_fc.py":[{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"dabe24b05e6717c7225cf2e1ec32daddaa23363c","unresolved":true,"context_lines":[{"line_number":56,"context_line":"            LOG.error(\u0027pywbem could not be imported! \u0027"},{"line_number":57,"context_line":"                      \u0027pywbem is necessary for this volume driver.\u0027)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"    @utils.trace"},{"line_number":60,"context_line":"    def create_volume(self, volume):"},{"line_number":61,"context_line":"        \"\"\"Create volume.\"\"\""},{"line_number":62,"context_line":"        model_update \u003d self.common.create_volume(volume)"}],"source_content_type":"text/x-python","patch_set":9,"id":"61ffa1f7_191f67ab","line":59,"updated":"2022-07-05 20:07:29.000000000","message":"trace method was moved to volume_utils:\n  https://review.opendev.org/c/openstack/cinder/+/772582\nIt is necessary to update here and the others to use the moved method.","commit_id":"bc30ede9423373cc3336dc30657bcca6c5e14df7"}],"cinder/volume/drivers/fujitsu/eternus_dx/eternus_dx_iscsi.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"b9f53060a7ee290b7fa08b295e77d48584a4f50d","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def create_volume(self, volume):"},{"line_number":58,"context_line":"        \"\"\"Create volume.\"\"\""},{"line_number":59,"context_line":"        LOG.debug(\u0027create_volume, volume id: %s, Enter method.\u0027, volume[\u0027id\u0027])"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        model_update \u003d self.common.create_volume(volume)"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1fa4df85_e9d49aeb","line":59,"updated":"2020-03-17 12:05:31.000000000","message":"you can replace these entry/exit logging  LOG.debug with cinder.volume.utils trace decorator\n\n@utils.trace","commit_id":"945efec4a0d49fd8dca3f14e1dc009e1bf79246a"}]}