)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"356eb6eed2ce0ccaf837259930d1f3db944c28f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1adbf907_8ac6ec5d","updated":"2023-08-01 12:31:26.000000000","message":"run-Yadro Tatlin Unified CI","commit_id":"8f1eb75c7844dce92d78985cf93b72dc1ae9eb44"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"df0c2a6a46098f49a25793da1ebf33471ade286c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1fda7d1c_6a2e8986","updated":"2023-08-23 11:27:47.000000000","message":"Thanks Gorka!","commit_id":"5b4ab0027e56997585c1a1ba7e19f7ddea722fa6"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"88f9470d6aeb238b4734c2544d4f256f4a422b81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"91687c2f_b67a4b1a","updated":"2023-08-23 16:00:32.000000000","message":"That\u0027s a nasty bug.  Glad to see it fixed.  Code and tests LGTM.","commit_id":"5b4ab0027e56997585c1a1ba7e19f7ddea722fa6"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"43d03560dd14042ba6618cabdcc88e14cd8199b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"53024b02_64c145bf","updated":"2023-08-24 01:59:52.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican :  tempest.api.compute.servers.test_server_rescue.ServerStableDeviceRescueTest.test_stable_device_rescue_disk_virtio_with_volume_attached hit a server fault","commit_id":"5b4ab0027e56997585c1a1ba7e19f7ddea722fa6"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ec8574b3b8830bac7693543f889859334a76e053","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a271fddd_6e87b093","updated":"2023-08-23 22:27:05.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican : test_incr_backup_create_and_restore_to_an_existing_volume - volume went to error status","commit_id":"5b4ab0027e56997585c1a1ba7e19f7ddea722fa6"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"836231c9d06b2e9ed16cf90421f29317ffa0e4ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"233baa57_c77d84d5","updated":"2023-08-23 19:42:45.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican bunch of 500s from the identity service","commit_id":"5b4ab0027e56997585c1a1ba7e19f7ddea722fa6"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ab633efcdf49d2123f28510bef84463ebddcc5bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ff3bbc1d_bc60b4aa","updated":"2023-08-23 15:39:18.000000000","message":"recheck unrelated failures in os-brick-src-tempest-lvm-lio-barbican","commit_id":"5b4ab0027e56997585c1a1ba7e19f7ddea722fa6"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"7984dc983b5407d7700f4cc5d10f1b35795921de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6edaf9cc_8d981b12","updated":"2023-08-25 08:53:10.000000000","message":"Brian already +2 the previous patch and in the last patchset the only change is the nvme hostid tests added by other patch.","commit_id":"1435059da5c3b66e83ec9108cd85d17471a8d747"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"c0b1c356879e320bf18eb18a27449541741246a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f2f2acd7_4dad63a9","updated":"2023-08-24 14:30:27.000000000","message":"recheck","commit_id":"1435059da5c3b66e83ec9108cd85d17471a8d747"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"900e1150057c21842f613ba7dd7405e6ed0df652","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5ab98f5c_0a6fd296","updated":"2023-08-24 18:01:56.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican\n\ntempest.scenario.test_volume_boot_pattern.TestVolumeBootPattern\t\nurllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host\u003d\u0027158.69.72.48\u0027, port\u003d443): Read timed out. (read timeout\u003d60)","commit_id":"1435059da5c3b66e83ec9108cd85d17471a8d747"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"94a08520942a39a1b40f06863bf4515b19405ffb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b3ddafcc_04d1d25b","updated":"2023-08-25 12:07:08.000000000","message":"recheck unrelated failure in build-openstack-releasenotes","commit_id":"1435059da5c3b66e83ec9108cd85d17471a8d747"}],"os_brick/tests/initiator/test_linuxscsi.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d9d60ec9d1dea5a2d4afd58e74f86b7603138f3a","unresolved":true,"context_lines":[{"line_number":987,"context_line":"            exception.BrickException,"},{"line_number":988,"context_line":"            self.linuxscsi.extend_volume,"},{"line_number":989,"context_line":"            [\u0027/dev/fake1\u0027, \u0027/dev/fake2\u0027],"},{"line_number":990,"context_line":"            use_multipath\u003dTrue)"},{"line_number":991,"context_line":""},{"line_number":992,"context_line":"    def test_process_lun_id_list(self):"},{"line_number":993,"context_line":"        lun_list \u003d [2, 255, 88, 370, 5, 256]"}],"source_content_type":"text/x-python","patch_set":3,"id":"eb990468_4e6a3d23","line":990,"updated":"2023-08-23 10:55:42.000000000","message":"-1: You must check that mock_valid_dev is called with the expected parameters","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"df0c2a6a46098f49a25793da1ebf33471ade286c","unresolved":false,"context_lines":[{"line_number":987,"context_line":"            exception.BrickException,"},{"line_number":988,"context_line":"            self.linuxscsi.extend_volume,"},{"line_number":989,"context_line":"            [\u0027/dev/fake1\u0027, \u0027/dev/fake2\u0027],"},{"line_number":990,"context_line":"            use_multipath\u003dTrue)"},{"line_number":991,"context_line":""},{"line_number":992,"context_line":"    def test_process_lun_id_list(self):"},{"line_number":993,"context_line":"        lun_list \u003d [2, 255, 88, 370, 5, 256]"}],"source_content_type":"text/x-python","patch_set":3,"id":"7c89a3c9_6e331c70","line":990,"in_reply_to":"eb990468_4e6a3d23","updated":"2023-08-23 11:27:47.000000000","message":"Done","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"}],"os_brick/tests/test_utils.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d9d60ec9d1dea5a2d4afd58e74f86b7603138f3a","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        self.assertEqual(expected, is_valid)"},{"line_number":55,"context_line":"        mock_execute._execute.assert_called_once_with("},{"line_number":56,"context_line":"            \u0027dd\u0027, \u0027if\u003d/dev/fake\u0027, \u0027of\u003d/dev/null\u0027, \u0027count\u003d1\u0027,"},{"line_number":57,"context_line":"            run_as_root\u003dTrue, root_helper\u003dmock_execute._root_helper)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"class TestRetryDecorator(base.TestCase):"}],"source_content_type":"text/x-python","patch_set":3,"id":"d106fd37_c5af1800","line":57,"updated":"2023-08-23 10:55:42.000000000","message":"-1: There are 3 code paths in the check_valid_device method, but we are only testing one.","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"df0c2a6a46098f49a25793da1ebf33471ade286c","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        self.assertEqual(expected, is_valid)"},{"line_number":55,"context_line":"        mock_execute._execute.assert_called_once_with("},{"line_number":56,"context_line":"            \u0027dd\u0027, \u0027if\u003d/dev/fake\u0027, \u0027of\u003d/dev/null\u0027, \u0027count\u003d1\u0027,"},{"line_number":57,"context_line":"            run_as_root\u003dTrue, root_helper\u003dmock_execute._root_helper)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"class TestRetryDecorator(base.TestCase):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7d3a0c19_f6f5fe9f","line":57,"in_reply_to":"d106fd37_c5af1800","updated":"2023-08-23 11:27:47.000000000","message":"Done","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"}],"os_brick/utils.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d9d60ec9d1dea5a2d4afd58e74f86b7603138f3a","unresolved":true,"context_lines":[{"line_number":426,"context_line":"                  {\"path\": path, \"error\": e.stderr})"},{"line_number":427,"context_line":"        return False"},{"line_number":428,"context_line":"    # If the info is none, the path does not exist."},{"line_number":429,"context_line":"    if info is None:"},{"line_number":430,"context_line":"        return False"},{"line_number":431,"context_line":"    return True"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"class Anything(object):"}],"source_content_type":"text/x-python","patch_set":3,"id":"2a67e7b8_e81715a2","line":431,"range":{"start_line":429,"start_character":0,"end_line":431,"end_character":15},"updated":"2023-08-23 10:55:42.000000000","message":"nit: Can be expressed as a oneliner\n\n   return info is not None","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"df0c2a6a46098f49a25793da1ebf33471ade286c","unresolved":false,"context_lines":[{"line_number":426,"context_line":"                  {\"path\": path, \"error\": e.stderr})"},{"line_number":427,"context_line":"        return False"},{"line_number":428,"context_line":"    # If the info is none, the path does not exist."},{"line_number":429,"context_line":"    if info is None:"},{"line_number":430,"context_line":"        return False"},{"line_number":431,"context_line":"    return True"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"class Anything(object):"}],"source_content_type":"text/x-python","patch_set":3,"id":"e044ac4b_1fbccdb8","line":431,"range":{"start_line":429,"start_character":0,"end_line":431,"end_character":15},"in_reply_to":"2a67e7b8_e81715a2","updated":"2023-08-23 11:27:47.000000000","message":"Done","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"}],"releasenotes/notes/fix-extend-multipath-a308d333061665fd.yaml":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d9d60ec9d1dea5a2d4afd58e74f86b7603138f3a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    `Bug #2032177 \u003chttps://bugs.launchpad.net/os-brick/+bug/2032177\u003e`_:"},{"line_number":5,"context_line":"    Handled the issue when one or more paths of a multipath device are down and"},{"line_number":6,"context_line":"    we try to extend the device. This is addressed by adding an additional check"},{"line_number":7,"context_line":"    to verify that all devices are up before extending the multipath device."},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"b0008a1c_f02d891d","line":7,"updated":"2023-08-23 10:55:42.000000000","message":"nit: Mention that previously it was leaving devices in an inconsistent state?  For example adding at the end: \"to avoid leaving devices with inconsistent sizes.\"","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"df0c2a6a46098f49a25793da1ebf33471ade286c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    `Bug #2032177 \u003chttps://bugs.launchpad.net/os-brick/+bug/2032177\u003e`_:"},{"line_number":5,"context_line":"    Handled the issue when one or more paths of a multipath device are down and"},{"line_number":6,"context_line":"    we try to extend the device. This is addressed by adding an additional check"},{"line_number":7,"context_line":"    to verify that all devices are up before extending the multipath device."},{"line_number":8,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"843bc7ee_aac8495d","line":7,"in_reply_to":"b0008a1c_f02d891d","updated":"2023-08-23 11:27:47.000000000","message":"Done","commit_id":"c5e97671968ce24b77da717d5078ce41cf39a112"}]}