)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38939,"name":"Roger Koranteng","display_name":"rogerkoranteng","email":"rogerkorantenng@gmail.com","username":"rogerkoranteng"},"change_message_id":"2e8860f6fe2efaefaf9bfa4f7cfa8b05d27c9fc2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"236d2ab0_30209650","updated":"2026-03-27 22:46:30.000000000","message":"recheck","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"},{"author":{"_account_id":38939,"name":"Roger Koranteng","display_name":"rogerkoranteng","email":"rogerkorantenng@gmail.com","username":"rogerkoranteng"},"change_message_id":"0597afe664837c2fd8234ff30c0472a0b41d3cf6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a3eaf9c6_89dfcf8e","in_reply_to":"236d2ab0_30209650","updated":"2026-03-31 22:47:25.000000000","message":"recheck","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"546cadd444bdf7298022ff0f78612af15ea4211b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9d402b7d_6223e67c","updated":"2026-05-04 18:56:21.000000000","message":"LGTM, thank you!","commit_id":"a50632baeacfb6a6d2224b3d7e8dc174498a876f"}],"manila/share/drivers/hitachi/hsp/rest.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1e88f02837b87823a1f39c8c4a7eb892de80c727","unresolved":true,"context_lines":[{"line_number":37,"context_line":"            self.verify \u003d False"},{"line_number":38,"context_line":"            requests.packages.urllib3.disable_warnings("},{"line_number":39,"context_line":"                requests.packages.urllib3.exceptions.InsecureRequestWarning)"},{"line_number":40,"context_line":"            LOG.warning(\"SSL certificate verification is disabled.\")"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def _send_post(self, url, payload\u003dNone):"},{"line_number":43,"context_line":"        resp \u003d requests.post(url, auth\u003d(self.username, self.password),"}],"source_content_type":"text/x-python","patch_set":1,"id":"d6c37607_04695771","line":40,"updated":"2026-03-31 21:15:26.000000000","message":"Consider including the backend host in the warning so operators can\nidentify which backend has verification disabled when multiple\nbackends are configured.","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"},{"author":{"_account_id":38939,"name":"Roger Koranteng","display_name":"rogerkoranteng","email":"rogerkorantenng@gmail.com","username":"rogerkoranteng"},"change_message_id":"0597afe664837c2fd8234ff30c0472a0b41d3cf6","unresolved":false,"context_lines":[{"line_number":37,"context_line":"            self.verify \u003d False"},{"line_number":38,"context_line":"            requests.packages.urllib3.disable_warnings("},{"line_number":39,"context_line":"                requests.packages.urllib3.exceptions.InsecureRequestWarning)"},{"line_number":40,"context_line":"            LOG.warning(\"SSL certificate verification is disabled.\")"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def _send_post(self, url, payload\u003dNone):"},{"line_number":43,"context_line":"        resp \u003d requests.post(url, auth\u003d(self.username, self.password),"}],"source_content_type":"text/x-python","patch_set":1,"id":"38a62531_7631d7dd","line":40,"in_reply_to":"b3960a9f_525c4cd3","updated":"2026-03-31 22:47:25.000000000","message":"A new patch 2 has been pushed","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"},{"author":{"_account_id":38939,"name":"Roger Koranteng","display_name":"rogerkoranteng","email":"rogerkorantenng@gmail.com","username":"rogerkoranteng"},"change_message_id":"e83139586fe7880cfd4a72de6bdbc9436b7a40b7","unresolved":true,"context_lines":[{"line_number":37,"context_line":"            self.verify \u003d False"},{"line_number":38,"context_line":"            requests.packages.urllib3.disable_warnings("},{"line_number":39,"context_line":"                requests.packages.urllib3.exceptions.InsecureRequestWarning)"},{"line_number":40,"context_line":"            LOG.warning(\"SSL certificate verification is disabled.\")"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def _send_post(self, url, payload\u003dNone):"},{"line_number":43,"context_line":"        resp \u003d requests.post(url, auth\u003d(self.username, self.password),"}],"source_content_type":"text/x-python","patch_set":1,"id":"b3960a9f_525c4cd3","line":40,"in_reply_to":"d6c37607_04695771","updated":"2026-03-31 22:37:03.000000000","message":"Acknowledged. Will use %s with oslo.log lazy formatting to pass hsp_host into the warning, so each backend is identifiable in multi-backend setups.","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"}],"manila/tests/share/drivers/hitachi/hsp/test_rest.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1e88f02837b87823a1f39c8c4a7eb892de80c727","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    def test__send_post(self, code):"},{"line_number":57,"context_line":"        self.mock_object(requests, \"post\", mock.Mock("},{"line_number":58,"context_line":"            return_value\u003dFakeRequests(code)))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        if code \u003d\u003d 202:"},{"line_number":61,"context_line":"            self.mock_object(rest.HSPRestBackend, \"_wait_job_status\","},{"line_number":62,"context_line":"                             mock.Mock())"}],"source_content_type":"text/x-python","patch_set":1,"id":"cbe3acfe_18c48586","line":59,"updated":"2026-03-31 21:15:26.000000000","message":"The requests.post.assert_called_once_with(...) assertion in\ntest__send_post is placed BEFORE self._driver._send_post(\u0027fake_url\u0027)\nis called. At this point, requests.post has been mocked but never\ninvoked, so the assertion checks a mock that hasn\u0027t been called.\nMove the assertion to AFTER the _send_post call. The same issue\napplies to test__send_get and test__send_delete.","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"},{"author":{"_account_id":38939,"name":"Roger Koranteng","display_name":"rogerkoranteng","email":"rogerkorantenng@gmail.com","username":"rogerkoranteng"},"change_message_id":"e83139586fe7880cfd4a72de6bdbc9436b7a40b7","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    def test__send_post(self, code):"},{"line_number":57,"context_line":"        self.mock_object(requests, \"post\", mock.Mock("},{"line_number":58,"context_line":"            return_value\u003dFakeRequests(code)))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        if code \u003d\u003d 202:"},{"line_number":61,"context_line":"            self.mock_object(rest.HSPRestBackend, \"_wait_job_status\","},{"line_number":62,"context_line":"                             mock.Mock())"}],"source_content_type":"text/x-python","patch_set":1,"id":"f4fc60a1_9afa8c6e","line":59,"in_reply_to":"cbe3acfe_18c48586","updated":"2026-03-31 22:37:03.000000000","message":"Noted. Will move assert_called_once_with() after the method calls in all three tests so the mock is verified post-invocation.","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"},{"author":{"_account_id":38939,"name":"Roger Koranteng","display_name":"rogerkoranteng","email":"rogerkorantenng@gmail.com","username":"rogerkoranteng"},"change_message_id":"0597afe664837c2fd8234ff30c0472a0b41d3cf6","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    def test__send_post(self, code):"},{"line_number":57,"context_line":"        self.mock_object(requests, \"post\", mock.Mock("},{"line_number":58,"context_line":"            return_value\u003dFakeRequests(code)))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        if code \u003d\u003d 202:"},{"line_number":61,"context_line":"            self.mock_object(rest.HSPRestBackend, \"_wait_job_status\","},{"line_number":62,"context_line":"                             mock.Mock())"}],"source_content_type":"text/x-python","patch_set":1,"id":"c636fdf8_52cc1ad7","line":59,"in_reply_to":"f4fc60a1_9afa8c6e","updated":"2026-03-31 22:47:25.000000000","message":"A new patch 2 has been pushed.","commit_id":"7243483babab231480988080fd8ec897d6f82fbd"}]}