)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cb08ade08e93698e1bc57502e931dd23e43a8134","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"46c0964f_e021b201","updated":"2024-02-03 21:26:05.000000000","message":"One question inline, but otherwise LGTM.","commit_id":"60b63866ee06591041deec0c825fa531f2f8c462"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"739a09a4758eca2df318d51b4321dbb25fd4871d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bca914ef_543e4316","updated":"2024-02-05 18:49:17.000000000","message":"Revisions LGTM.","commit_id":"a31e641d73a0c1d8a8cb2701bab64663514ece44"}],"cinder/zonemanager/drivers/brocade/brcd_rest_fc_zone_client.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cb08ade08e93698e1bc57502e931dd23e43a8134","unresolved":true,"context_lines":[{"line_number":120,"context_line":"                                    rest_constants.AUTHORIZATION: auth}"},{"line_number":121,"context_line":"        else:"},{"line_number":122,"context_line":"            msg \u003d (_(\"REST login failed: %s\")"},{"line_number":123,"context_line":"                   % str(response.text))"},{"line_number":124,"context_line":"            LOG.error(msg)"},{"line_number":125,"context_line":"            raise exception.BrocadeZoningRestException(reason\u003dmsg)"},{"line_number":126,"context_line":"        return response.status_code"}],"source_content_type":"text/x-python","patch_set":2,"id":"7ee98eab_cdaeaca0","line":123,"range":{"start_line":123,"start_character":21,"end_line":123,"end_character":39},"updated":"2024-02-03 21:26:05.000000000","message":"I suspect that this is redundant and you don\u0027t need to call the str function here because the string interpolation operation will use the __str__ of whatever is passed.","commit_id":"60b63866ee06591041deec0c825fa531f2f8c462"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"671d1e15e6e916bb4848457d49dc5b9c05c084d1","unresolved":false,"context_lines":[{"line_number":120,"context_line":"                                    rest_constants.AUTHORIZATION: auth}"},{"line_number":121,"context_line":"        else:"},{"line_number":122,"context_line":"            msg \u003d (_(\"REST login failed: %s\")"},{"line_number":123,"context_line":"                   % str(response.text))"},{"line_number":124,"context_line":"            LOG.error(msg)"},{"line_number":125,"context_line":"            raise exception.BrocadeZoningRestException(reason\u003dmsg)"},{"line_number":126,"context_line":"        return response.status_code"}],"source_content_type":"text/x-python","patch_set":2,"id":"99b0c74c_edcf84b9","line":123,"range":{"start_line":123,"start_character":21,"end_line":123,"end_character":39},"in_reply_to":"7ee98eab_cdaeaca0","updated":"2024-02-04 06:06:03.000000000","message":"+1. Done","commit_id":"60b63866ee06591041deec0c825fa531f2f8c462"}]}