)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"caafaddf8adf43ebe092de5c34ced35e0cc67801","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"415f0acd_bcecf66b","updated":"2023-07-05 04:22:28.000000000","message":"looks good","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"c631477cdf61c9cf8a1955a6621837dddab7316d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"521f51b7_83dce47c","updated":"2023-06-12 09:14:20.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican failures","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"5483f5169b086c14b1491d9d8c9845d31c20ec44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"069ab825_0af2401f","updated":"2023-07-17 13:19:33.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"d1fc9235062957ef3e68ae388f814c84b1bb9fbd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0c74a629_3ba4634c","updated":"2023-06-15 08:04:17.000000000","message":"run-DellEMC PowerStore CI","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"f88a78e3969c0a6a54d8a97c6bc5b1c3fe4b5988","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2f509a10_459d54ec","updated":"2023-06-15 02:58:39.000000000","message":"run-DellEMC PowerStore CI","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"7aeeb3fb312fdb3865d93b52a3bd366089ce4225","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9258f924_4bc9f4e2","updated":"2023-06-15 06:42:25.000000000","message":"run-DellEMC PowerStore CI","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"53aeae6f69e60153c42e49d7bf3b7c26dfb01207","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"93c0580b_3d331bfc","updated":"2023-06-15 06:44:14.000000000","message":"run-DellEMC PowerStore CI","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"cc64675377d055123bf3a73e221c1d67167cd64f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"abc4b078_2b26ca50","updated":"2023-06-12 13:36:40.000000000","message":"run-DellEMC PowerStore CI","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"51fcfe319b39835a8acbe466061e9311915a5c63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c41d91ed_1e1d5ca2","updated":"2023-06-15 08:02:12.000000000","message":"run-DellEMC PowerStore CI","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"1d857a2c588d55099aef0508e50784189ef3ee09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"11778713_739f78c0","updated":"2023-09-20 05:28:10.000000000","message":"one update in releaenote otherwise looks good to me.","commit_id":"960695d720f8fdc11419d3c9b3eb6bdcf625ee55"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"b6dec1908c8bb6f60d24d31faefd89c20b939e9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a275bfeb_86c2e924","updated":"2023-09-20 10:50:22.000000000","message":"LGTM","commit_id":"42d9a892805225c13b87d5e68b6000fe01b7ae99"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"acd9e7187efb5715bced92679da33b3658b42899","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"370b6e10_b51e33ec","updated":"2023-09-21 06:47:04.000000000","message":"recheck","commit_id":"42d9a892805225c13b87d5e68b6000fe01b7ae99"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"47683fe63c3ebf89c3ffffcc3624b03c926e7c08","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e28b3a41_a3a5a974","updated":"2023-09-20 08:48:13.000000000","message":"recheck","commit_id":"42d9a892805225c13b87d5e68b6000fe01b7ae99"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"ff831cb6c5d04da1bae393285b97e0fa81ed57e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fb7c12c8_ba2773cc","updated":"2023-09-21 06:34:29.000000000","message":"recheck","commit_id":"42d9a892805225c13b87d5e68b6000fe01b7ae99"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"ff831cb6c5d04da1bae393285b97e0fa81ed57e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7cae329e_e037442a","in_reply_to":"e28b3a41_a3a5a974","updated":"2023-09-21 06:34:29.000000000","message":"recheck","commit_id":"42d9a892805225c13b87d5e68b6000fe01b7ae99"}],"os_brick/initiator/connectors/scaleio.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"a6f087d0edb50c57764ea79a3fe21a201c3e4afd","unresolved":true,"context_lines":[{"line_number":49,"context_line":"    OK_STATUS_CODE \u003d 200"},{"line_number":50,"context_line":"    VOLUME_NOT_MAPPED_ERROR \u003d 84"},{"line_number":51,"context_line":"    VOLUME_ALREADY_MAPPED_ERROR \u003d 81"},{"line_number":52,"context_line":"    VOLUME_ALREADY_MAPPED_ERROR_TOKYO \u003d 4037"},{"line_number":53,"context_line":"    GET_GUID_OP_CODE \u003d io(\u0027a\u0027, 14)"},{"line_number":54,"context_line":"    RESCAN_VOLS_OP_CODE \u003d io(\u0027a\u0027, 10)"},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"8ee2d1ce_8423c48b","line":52,"range":{"start_line":52,"start_character":32,"end_line":52,"end_character":37},"updated":"2023-09-15 14:17:31.000000000","message":"would\u0027ve been better to use some meaningful information instead of TOKYO (though i don\u0027t know if it\u0027s a code for conveying something)\n\nSomething like,\n\nVOLUME_ALREADY_MAPPED_ERROR_v4","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"af861f4637d056b8f4e30b591552c1cf3f71660c","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    OK_STATUS_CODE \u003d 200"},{"line_number":50,"context_line":"    VOLUME_NOT_MAPPED_ERROR \u003d 84"},{"line_number":51,"context_line":"    VOLUME_ALREADY_MAPPED_ERROR \u003d 81"},{"line_number":52,"context_line":"    VOLUME_ALREADY_MAPPED_ERROR_TOKYO \u003d 4037"},{"line_number":53,"context_line":"    GET_GUID_OP_CODE \u003d io(\u0027a\u0027, 14)"},{"line_number":54,"context_line":"    RESCAN_VOLS_OP_CODE \u003d io(\u0027a\u0027, 10)"},{"line_number":55,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"e1b2e536_e21575dd","line":52,"range":{"start_line":52,"start_character":32,"end_line":52,"end_character":37},"in_reply_to":"8ee2d1ce_8423c48b","updated":"2023-09-20 03:26:28.000000000","message":"Has modified the VOLUME_ALREADY_MAPPED_ERROR_TOKYO into VOLUME_ALREADY_MAPPED_ERROR_v4.","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"}],"os_brick/tests/initiator/connectors/test_scaleio.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"a6f087d0edb50c57764ea79a3fe21a201c3e4afd","unresolved":true,"context_lines":[{"line_number":304,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":305,"context_line":"                          self.test_disconnect_volume)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def test_error_disconnect_volume_tokyo(self):"},{"line_number":308,"context_line":"        \"\"\"Fail to disconnect with REST API failure\"\"\""},{"line_number":309,"context_line":"        self.mock_calls[self.action_format.format("},{"line_number":310,"context_line":"            \u0027removeMappedSdc\u0027)] \u003d self.MockHTTPSResponse("},{"line_number":311,"context_line":"            dict(errorCode\u003dself.connector.VOLUME_ALREADY_MAPPED_ERROR_TOKYO,"},{"line_number":312,"context_line":"                 message\u003d\u0027Test error map volume\u0027), 500)"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":315,"context_line":"                          self.test_disconnect_volume)"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    def test_disconnect_volume_not_mapped(self):"},{"line_number":318,"context_line":"        \"\"\"Ignore REST API failure for volume not mapped\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1e32249f_ee7ec966","line":315,"range":{"start_line":307,"start_character":0,"end_line":315,"end_character":54},"updated":"2023-09-15 14:17:31.000000000","message":"maybe I\u0027m missing something here but this test doesn\u0027t make sense to me.\nWhen is the case where we will get VOLUME_ALREADY_MAPPED_ERROR during disconnect? Isn\u0027t it a failure case when attaching a volume and not detaching?\n\nAlso not sure how this test passes ...","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"1252d4b4ac909b8bb56f178f0cb2c5b21e2f6c05","unresolved":true,"context_lines":[{"line_number":304,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":305,"context_line":"                          self.test_disconnect_volume)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def test_error_disconnect_volume_tokyo(self):"},{"line_number":308,"context_line":"        \"\"\"Fail to disconnect with REST API failure\"\"\""},{"line_number":309,"context_line":"        self.mock_calls[self.action_format.format("},{"line_number":310,"context_line":"            \u0027removeMappedSdc\u0027)] \u003d self.MockHTTPSResponse("},{"line_number":311,"context_line":"            dict(errorCode\u003dself.connector.VOLUME_ALREADY_MAPPED_ERROR_TOKYO,"},{"line_number":312,"context_line":"                 message\u003d\u0027Test error map volume\u0027), 500)"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":315,"context_line":"                          self.test_disconnect_volume)"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    def test_disconnect_volume_not_mapped(self):"},{"line_number":318,"context_line":"        \"\"\"Ignore REST API failure for volume not mapped\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"6d8f0e56_25f7a2b1","line":315,"range":{"start_line":307,"start_character":0,"end_line":315,"end_character":54},"in_reply_to":"1e32249f_ee7ec966","updated":"2023-09-15 14:45:13.000000000","message":"I think that this test was based on the previous on which covered versions up to v4 (Tokyo). I agree that it doesn\u0027t make sense to test this scenario. Should we remove it?","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"8ed3ed5dfe466c149cd23e216e862057725d88c0","unresolved":true,"context_lines":[{"line_number":304,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":305,"context_line":"                          self.test_disconnect_volume)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def test_error_disconnect_volume_tokyo(self):"},{"line_number":308,"context_line":"        \"\"\"Fail to disconnect with REST API failure\"\"\""},{"line_number":309,"context_line":"        self.mock_calls[self.action_format.format("},{"line_number":310,"context_line":"            \u0027removeMappedSdc\u0027)] \u003d self.MockHTTPSResponse("},{"line_number":311,"context_line":"            dict(errorCode\u003dself.connector.VOLUME_ALREADY_MAPPED_ERROR_TOKYO,"},{"line_number":312,"context_line":"                 message\u003d\u0027Test error map volume\u0027), 500)"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":315,"context_line":"                          self.test_disconnect_volume)"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    def test_disconnect_volume_not_mapped(self):"},{"line_number":318,"context_line":"        \"\"\"Ignore REST API failure for volume not mapped\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"f2d50e64_485d965e","line":315,"range":{"start_line":307,"start_character":0,"end_line":315,"end_character":54},"in_reply_to":"6d8f0e56_25f7a2b1","updated":"2023-09-20 03:24:47.000000000","message":"I also thing the case is not necessary. Deleted this scenario.","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"81539bbb012b5ff9ab11ea97c36f0df3f90f0c0d","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":305,"context_line":"                          self.test_disconnect_volume)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    def test_error_disconnect_volume_tokyo(self):"},{"line_number":308,"context_line":"        \"\"\"Fail to disconnect with REST API failure\"\"\""},{"line_number":309,"context_line":"        self.mock_calls[self.action_format.format("},{"line_number":310,"context_line":"            \u0027removeMappedSdc\u0027)] \u003d self.MockHTTPSResponse("},{"line_number":311,"context_line":"            dict(errorCode\u003dself.connector.VOLUME_ALREADY_MAPPED_ERROR_TOKYO,"},{"line_number":312,"context_line":"                 message\u003d\u0027Test error map volume\u0027), 500)"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"        self.assertRaises(exception.BrickException,"},{"line_number":315,"context_line":"                          self.test_disconnect_volume)"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    def test_disconnect_volume_not_mapped(self):"},{"line_number":318,"context_line":"        \"\"\"Ignore REST API failure for volume not mapped\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"c4ca780d_783b88c3","line":315,"range":{"start_line":307,"start_character":0,"end_line":315,"end_character":54},"in_reply_to":"f2d50e64_485d965e","updated":"2023-09-20 03:27:45.000000000","message":"Resolved","commit_id":"57b99a8eb0ebf2fa0b78623fce148a02668a9c97"}],"releasenotes/notes/bug-2013749-3de9f827b82116a2.yaml":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"1d857a2c588d55099aef0508e50784189ef3ee09","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    PowerFlex driver `Bug #2013749"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/os-brick/+bug/2013749\u003e`_: Added"},{"line_number":6,"context_line":"    new error code for powerflex 4.x which was causeing this error. Powerflex"},{"line_number":7,"context_line":"    changed VOLUME_ALREADY_MAPPED_ERROR error to 4037"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b49d0456_65fd373d","line":6,"range":{"start_line":6,"start_character":47,"end_line":6,"end_character":55},"updated":"2023-09-20 05:28:10.000000000","message":"causing","commit_id":"960695d720f8fdc11419d3c9b3eb6bdcf625ee55"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"2d07dcf52c9d148d696653825e09b93e071b83ac","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    PowerFlex driver `Bug #2013749"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/os-brick/+bug/2013749\u003e`_: Added"},{"line_number":6,"context_line":"    new error code for powerflex 4.x which was causeing this error. Powerflex"},{"line_number":7,"context_line":"    changed VOLUME_ALREADY_MAPPED_ERROR error to 4037"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9df3964a_f1978802","line":6,"range":{"start_line":6,"start_character":47,"end_line":6,"end_character":55},"in_reply_to":"b49d0456_65fd373d","updated":"2023-09-20 06:51:58.000000000","message":"Done","commit_id":"960695d720f8fdc11419d3c9b3eb6bdcf625ee55"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"1d857a2c588d55099aef0508e50784189ef3ee09","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    PowerFlex driver `Bug #2013749"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/os-brick/+bug/2013749\u003e`_: Added"},{"line_number":6,"context_line":"    new error code for powerflex 4.x which was causeing this error. Powerflex"},{"line_number":7,"context_line":"    changed VOLUME_ALREADY_MAPPED_ERROR error to 4037"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ef4f2f3b_fa275e13","line":8,"range":{"start_line":5,"start_character":58,"end_line":8,"end_character":0},"updated":"2023-09-20 05:28:10.000000000","message":"This can be rewritten as,\n\nAdded handling for the new error code returned from Powerflex 4.x when a we try to map a volume which is already mapped.","commit_id":"960695d720f8fdc11419d3c9b3eb6bdcf625ee55"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"2d07dcf52c9d148d696653825e09b93e071b83ac","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    PowerFlex driver `Bug #2013749"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/os-brick/+bug/2013749\u003e`_: Added"},{"line_number":6,"context_line":"    new error code for powerflex 4.x which was causeing this error. Powerflex"},{"line_number":7,"context_line":"    changed VOLUME_ALREADY_MAPPED_ERROR error to 4037"},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8c934fe3_85c74696","line":8,"range":{"start_line":5,"start_character":58,"end_line":8,"end_character":0},"in_reply_to":"ef4f2f3b_fa275e13","updated":"2023-09-20 06:51:58.000000000","message":"Done","commit_id":"960695d720f8fdc11419d3c9b3eb6bdcf625ee55"}]}
