)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d95029ef7057d0803933c090ed23736a450e0ae9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2606f3ae_6b522faa","updated":"2026-03-18 19:35:16.000000000","message":"recheck\n\ndoc website was down, couldn\u0027t grab the constraints in time","commit_id":"b43b921135529c65992d5ee546685617b0562558"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"841f5e6df4dc69ae6ceabcd045360d375b8bb521","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"936d1bbe_863943b9","updated":"2026-08-20 15:41:26.000000000","message":"I\u0027ve a feeling failures are related to this change - could you please look into it?","commit_id":"683eaa37f5cf1e363ecd6fdfee8490af4ba6d415"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"0ae6a5bef9cac9d98408a0c9dc6f00472ca0a425","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a179a428_07b6f925","updated":"2026-08-20 13:51:33.000000000","message":"LGTM. One minor comment on UT coverage.","commit_id":"683eaa37f5cf1e363ecd6fdfee8490af4ba6d415"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"07cdfac09717bc74fb5ebcdcca99d36a64856b79","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dfb84437_b94f4f87","updated":"2026-08-26 16:53:03.000000000","message":"LGTM","commit_id":"c1ff7114100b3d589e89d17b46053262e19e3b3c"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"5ea14412096ef072f6adb84e1880d8b14503fe61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"244b1b28_687c170b","updated":"2026-08-25 14:00:28.000000000","message":"LGTM, thank you for working on this change. CI is happy, so that makes me even more confident.","commit_id":"c1ff7114100b3d589e89d17b46053262e19e3b3c"}],"manilaclient/common/httpclient.py":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"0ae6a5bef9cac9d98408a0c9dc6f00472ca0a425","unresolved":true,"context_lines":[{"line_number":123,"context_line":"    def delete(self, url, **kwargs):"},{"line_number":124,"context_line":"        return self._cs_request(url, \u0027DELETE\u0027, **kwargs)"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    def get_with_base_url(self, url, **kwargs):"},{"line_number":127,"context_line":"        base_url \u003d self._get_base_url(self.endpoint_override)"},{"line_number":128,"context_line":"        return self._cs_request("},{"line_number":129,"context_line":"            base_url + url,"}],"source_content_type":"text/x-python","patch_set":2,"id":"933009a4_f0618e64","line":126,"range":{"start_line":126,"start_character":8,"end_line":126,"end_character":25},"updated":"2026-08-20 13:51:33.000000000","message":"couldn\u0027t find a UT to address this method. May be its not required as _get_base_url already is covered? Please check once.","commit_id":"683eaa37f5cf1e363ecd6fdfee8490af4ba6d415"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d0d3710f85cfc260714df11c50a6d80d552acff3","unresolved":false,"context_lines":[{"line_number":123,"context_line":"    def delete(self, url, **kwargs):"},{"line_number":124,"context_line":"        return self._cs_request(url, \u0027DELETE\u0027, **kwargs)"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    def get_with_base_url(self, url, **kwargs):"},{"line_number":127,"context_line":"        base_url \u003d self._get_base_url(self.endpoint_override)"},{"line_number":128,"context_line":"        return self._cs_request("},{"line_number":129,"context_line":"            base_url + url,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ed606b62_dc491c5a","line":126,"range":{"start_line":126,"start_character":8,"end_line":126,"end_character":25},"in_reply_to":"933009a4_f0618e64","updated":"2026-08-24 20:16:22.000000000","message":"Honestly, you made me think this through. Thanks! Added test cases for the whole SessionClient here.","commit_id":"683eaa37f5cf1e363ecd6fdfee8490af4ba6d415"}]}