)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"a668256243ff7f8e0e0cf0965446606177cc1245","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0af812f8_cef72e2f","updated":"2026-07-16 10:20:06.000000000","message":"just a nit, if you need to respin.","commit_id":"adca10ced5ddd721153b7c5213b216fa88abd72d"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"6a46e933a81e82abb0fa29cea10e4cc9e938926b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"abb73820_5213f790","updated":"2026-07-08 16:53:45.000000000","message":"recheck","commit_id":"adca10ced5ddd721153b7c5213b216fa88abd72d"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"5099001cbad207a418f979e4d6f70f733c7fa7ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"98c03312_9a6f70b9","updated":"2026-07-16 10:04:33.000000000","message":"the best thing about this change is, its minimal and on point as per commit msg.\nlgtm.","commit_id":"adca10ced5ddd721153b7c5213b216fa88abd72d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2daebc3dfaa31e367323cdc6c883be548d704b69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b4c0683b_054bd484","updated":"2026-08-04 22:33:07.000000000","message":"LGTM.","commit_id":"e14bb95c72532e0b08b081789a793b783ed3c18e"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"1708fc3696f7c7014d4567fb79b22cb2b3d78df5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3f9858a3_aa34be89","updated":"2026-08-02 23:53:39.000000000","message":"Thanks for working on this. The solution looks good to me.","commit_id":"e14bb95c72532e0b08b081789a793b783ed3c18e"}],"cinder/tests/unit/test_service.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"a668256243ff7f8e0e0cf0965446606177cc1245","unresolved":true,"context_lines":[{"line_number":480,"context_line":"            self.assertEqual(value, getattr(cluster, key))"},{"line_number":481,"context_line":""},{"line_number":482,"context_line":""},{"line_number":483,"context_line":"@unittest.skipIf("},{"line_number":484,"context_line":"    utils.concurrency_mode_threading(),"},{"line_number":485,"context_line":"    \u0027oslo_service.wsgi.Server is incompatible with the threading backend; \u0027"},{"line_number":486,"context_line":"    \u0027when eventlet is disabled the API is deployed via an external WSGI \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"2b9b5de5_1abc3ea2","line":483,"range":{"start_line":483,"start_character":1,"end_line":483,"end_character":9},"updated":"2026-07-16 10:20:06.000000000","message":"nit: `test.testtools.skipIf` is used in most places, noticed in next patch in chain.","commit_id":"adca10ced5ddd721153b7c5213b216fa88abd72d"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"5099001cbad207a418f979e4d6f70f733c7fa7ea","unresolved":true,"context_lines":[{"line_number":486,"context_line":"    \u0027when eventlet is disabled the API is deployed via an external WSGI \u0027"},{"line_number":487,"context_line":"    \u0027server rather than WSGIService.\u0027)"},{"line_number":488,"context_line":"class TestWSGIService(test.TestCase):"},{"line_number":489,"context_line":""},{"line_number":490,"context_line":"    @mock.patch(\u0027oslo_service.wsgi.Loader\u0027)"},{"line_number":491,"context_line":"    def test_service_random_port(self, mock_loader):"},{"line_number":492,"context_line":"        test_service \u003d service.WSGIService(\"test_service\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"c89654c9_c27bfcb4","line":489,"updated":"2026-07-16 10:04:33.000000000","message":"yeah, all tests are inherntly eventlet-only from `service.WSGIService`, so its fine to skip at class level","commit_id":"adca10ced5ddd721153b7c5213b216fa88abd72d"}]}