)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"dd52706eea6bcb4334fa65eb73e152867c4a7739","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cd0f2ec7_a43e3b15","updated":"2024-02-02 15:39:13.000000000","message":"Suggestion inline.","commit_id":"3ae34499103058ca18fb90c2f85d3bf4f9e5bc66"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c934ae2ec9761695bea43a27264885eaba2524e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3efd16e6_ec514eb5","updated":"2024-02-01 16:29:46.000000000","message":"recheck the failures are unrelated","commit_id":"3ae34499103058ca18fb90c2f85d3bf4f9e5bc66"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5ea2a514712e0ef57f12dfe3b91967b03639cb47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6855c687_a2c7c017","updated":"2024-02-04 03:16:21.000000000","message":"LGTM.","commit_id":"86245ee7f9c5d6af7b201a8ba0b911a69d9553b1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"479905844caf678ac618aac3ef8c04307b5211ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f6d5c242_1aa9a316","updated":"2024-02-04 06:35:03.000000000","message":"recheck failure in tempest-ipv6-only is unrelated","commit_id":"86245ee7f9c5d6af7b201a8ba0b911a69d9553b1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6f3992999d0c5d3fcc3cf91c869b08c5b4356f28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7b770783_893a8269","updated":"2024-02-05 02:35:31.000000000","message":"recheck hedvig driver is not used in CI so the failure can\u0027t be related.","commit_id":"86245ee7f9c5d6af7b201a8ba0b911a69d9553b1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"3bffb53e2982d8f9585b3c53459216ea13293eaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a50acb90_941f9a02","updated":"2024-02-05 13:09:34.000000000","message":"recheck unrelated","commit_id":"86245ee7f9c5d6af7b201a8ba0b911a69d9553b1"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e09ca8225eb4eb959359460ea258df1307e22f51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d543d57b_d2363495","updated":"2024-02-04 16:19:57.000000000","message":"recheck unrelated failures","commit_id":"86245ee7f9c5d6af7b201a8ba0b911a69d9553b1"}],"cinder/volume/drivers/hedvig/rest_client.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"dd52706eea6bcb4334fa65eb73e152867c4a7739","unresolved":true,"context_lines":[{"line_number":113,"context_line":"                    self.nodeMap[node_] \u003d self.get_session_id(node_)"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"            except urllib.error.HTTPError as e:"},{"line_number":116,"context_line":"                if e.code \u003d\u003d http.client.NOT_FOUND:"},{"line_number":117,"context_line":"                    LOG.debug(\"Client not found\")"},{"line_number":118,"context_line":"                else:"},{"line_number":119,"context_line":"                    LOG.debug(\"Client not available\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"3d70cdfc_49f02d33","line":116,"updated":"2024-02-02 15:39:13.000000000","message":"Given that we only need the error code, it would be more lightweight to do\n\n    from http import HTTPStatus\n    \nat line 19 and then use HTTPStatus.NOT_FOUND here.","commit_id":"3ae34499103058ca18fb90c2f85d3bf4f9e5bc66"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9d0754dd742d469def0d15c51ff3e4f429609f39","unresolved":false,"context_lines":[{"line_number":113,"context_line":"                    self.nodeMap[node_] \u003d self.get_session_id(node_)"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"            except urllib.error.HTTPError as e:"},{"line_number":116,"context_line":"                if e.code \u003d\u003d http.client.NOT_FOUND:"},{"line_number":117,"context_line":"                    LOG.debug(\"Client not found\")"},{"line_number":118,"context_line":"                else:"},{"line_number":119,"context_line":"                    LOG.debug(\"Client not available\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"e7196e38_4d1c9f00","line":116,"in_reply_to":"3d70cdfc_49f02d33","updated":"2024-02-02 15:42:14.000000000","message":"It makes sense to me. I\u0027ve fixed this and will update the other patches containing the same","commit_id":"3ae34499103058ca18fb90c2f85d3bf4f9e5bc66"}]}
