)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"bdb0254bc47d6bf523856660ec2c6cee9e0e9e4f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6241d657_b2290ebd","updated":"2025-11-20 21:58:21.000000000","message":"Thanks for adding the tests! Verified that they are running and passing in Zuul.","commit_id":"2ea35f8374241db307d2e4db075a1c6a3a0a08af"}],"cinder/api/contrib/services.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"bdb0254bc47d6bf523856660ec2c6cee9e0e9e4f","unresolved":true,"context_lines":[{"line_number":137,"context_line":"                                      host, cluster_name)"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    # TODO: This currently returns HTTP 200 but it should return HTTP 204 since"},{"line_number":140,"context_line":"    # there\u0027s no content"},{"line_number":141,"context_line":"    @validation.schema(os_services.freeze_and_thaw)"},{"line_number":142,"context_line":"    def _thaw(self, req, context, body):"},{"line_number":143,"context_line":"        cluster_name, host \u003d common.get_cluster_host("}],"source_content_type":"text/x-python","patch_set":2,"id":"f70008b8_5a167734","line":140,"updated":"2025-11-20 21:58:21.000000000","message":"This is a really weird one because if an error occurs during the thaw host operation, the call at line 145 will return a string [0] ... I\u0027m not sure how that gets propagated out from the API.  (My comment here is about your comment, though, not the code change you\u0027re making.)\n\n[0] https://opendev.org/openstack/cinder/src/commit/5bc8ed7d63c53be56ac3ed245f3c94a463fb8e23/cinder/volume/api.py#L2334","commit_id":"2ea35f8374241db307d2e4db075a1c6a3a0a08af"}]}
