)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d3378d9375f96f3728a391b0e06852a89bede8a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4908d05c_0a2d8eae","updated":"2021-11-12 09:03:43.000000000","message":"Thanks Gorka.","commit_id":"4668ad6ee7c298cc8ea6e75391901007f58ec6b3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d162c609a1bdf0c1ad2f1c0872f5279c221b5a81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9e027ea8_c9f21b47","updated":"2021-11-12 15:19:19.000000000","message":"Addresses a bug that manifests in a not uncommon deployment configuration; appropriate to backport.  Approving a simultaneous backport to the stable branches because we\u0027re up against a deadline to do a final release from ussuri before it goes EM.  We will not be releasing from any other stable branches before the final ussuri release.","commit_id":"b03eca8353f1db7fed5dc630d225126b1edb1055"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"4a2eca5c6214f1975a06da9e876b7fcf1d07645f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e7c93e34_17eac5c1","updated":"2021-11-12 15:16:43.000000000","message":"LGTM","commit_id":"b03eca8353f1db7fed5dc630d225126b1edb1055"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"64e03814f67ff620ad0c7d4521428ce477564dc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1c90fa43_50ea92ca","updated":"2021-11-13 03:47:55.000000000","message":"recheck\n\ntempest-full-py3 timed out","commit_id":"b03eca8353f1db7fed5dc630d225126b1edb1055"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"9b851ae23078dd8dc56403c262661855d7194fcc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e98a1acf_7d8823d8","updated":"2021-11-12 19:00:34.000000000","message":"recheck os-brick-src-tempest-lvm-lio-barbican - failure is in cinder_tempest_plugin.api.volume.admin.test_volume_backup.VolumesBackupsTest.test_incremental_backup_respective_parents","commit_id":"b03eca8353f1db7fed5dc630d225126b1edb1055"}],"os_brick/initiator/connectors/iscsi.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"be03089caa147b6c4c6be1ac4d7359367f23e5b6","unresolved":true,"context_lines":[{"line_number":494,"context_line":"    @synchronized(\u0027connect_volume\u0027, external\u003dTrue)"},{"line_number":495,"context_line":"    def connect_volume(self, connection_properties):"},{"line_number":496,"context_line":"        \"\"\"Attach the volume to instance_name."},{"line_number":497,"context_line":""},{"line_number":498,"context_line":"        NOTE: Will retry up to three times to handle the case where c-vol"},{"line_number":499,"context_line":"        and n-cpu are both using os-brick to manage iSCSI sessions but they"},{"line_number":500,"context_line":"        are on the same node and using different locking directories. In this"},{"line_number":501,"context_line":"        case, even though this call is synchronized, they will be separate"},{"line_number":502,"context_line":"        locks and can still overlap with connect and disconnect. Since a"},{"line_number":503,"context_line":"        disconnect during an initial attach can\u0027t cause IO failure (the device"},{"line_number":504,"context_line":"        has not been made available yet), we just try the connection again."},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"        :param connection_properties: The valid dictionary that describes all"},{"line_number":507,"context_line":"                                      of the target volume attributes."}],"source_content_type":"text/x-python","patch_set":1,"id":"f787f6d7_355b2cae","line":504,"range":{"start_line":497,"start_character":0,"end_line":504,"end_character":75},"updated":"2021-11-12 08:57:27.000000000","message":"-1: The original patch removed this comment.","commit_id":"40ba2c8b805f50b7ef1b5fb9ee7d2a47052ab84b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d3378d9375f96f3728a391b0e06852a89bede8a3","unresolved":false,"context_lines":[{"line_number":494,"context_line":"    @synchronized(\u0027connect_volume\u0027, external\u003dTrue)"},{"line_number":495,"context_line":"    def connect_volume(self, connection_properties):"},{"line_number":496,"context_line":"        \"\"\"Attach the volume to instance_name."},{"line_number":497,"context_line":""},{"line_number":498,"context_line":"        NOTE: Will retry up to three times to handle the case where c-vol"},{"line_number":499,"context_line":"        and n-cpu are both using os-brick to manage iSCSI sessions but they"},{"line_number":500,"context_line":"        are on the same node and using different locking directories. In this"},{"line_number":501,"context_line":"        case, even though this call is synchronized, they will be separate"},{"line_number":502,"context_line":"        locks and can still overlap with connect and disconnect. Since a"},{"line_number":503,"context_line":"        disconnect during an initial attach can\u0027t cause IO failure (the device"},{"line_number":504,"context_line":"        has not been made available yet), we just try the connection again."},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"        :param connection_properties: The valid dictionary that describes all"},{"line_number":507,"context_line":"                                      of the target volume attributes."}],"source_content_type":"text/x-python","patch_set":1,"id":"ae5f2858_05d1dd6b","line":504,"range":{"start_line":497,"start_character":0,"end_line":504,"end_character":75},"in_reply_to":"f787f6d7_355b2cae","updated":"2021-11-12 09:03:43.000000000","message":"Done","commit_id":"40ba2c8b805f50b7ef1b5fb9ee7d2a47052ab84b"}]}
