)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2d17abf47fd8703780bcef3004574698e11e20fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ef89b31d_4219b4c9","updated":"2025-06-16 15:53:40.000000000","message":"Just an issue about left over lines from the previous way of testing","commit_id":"3a48741395a9a6c15002d6b491f5d033ea83066f"}],"glance/tests/functional/test_wsgi.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2d17abf47fd8703780bcef3004574698e11e20fa","unresolved":true,"context_lines":[{"line_number":115,"context_line":"        # think there are no valid images and deleting everything."},{"line_number":116,"context_line":"        # self.my_api_server.needs_database \u003d False"},{"line_number":117,"context_line":"        # self.start_with_retry(self.my_api_server,"},{"line_number":118,"context_line":"        #                       \u0027api_port\u0027, 3, **self.__dict__.copy())"},{"line_number":119,"context_line":"        self.start_server(run_staging_cleaner\u003dTrue)"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        # Poll to give it time to come up and do the work. Use the presence"}],"source_content_type":"text/x-python","patch_set":3,"id":"29ca9b8a_1930422f","line":118,"range":{"start_line":118,"start_character":9,"end_line":118,"end_character":32},"updated":"2025-06-16 15:53:40.000000000","message":"I think you can delete the commented lines?","commit_id":"3a48741395a9a6c15002d6b491f5d033ea83066f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"b077f2dd8c573a9b6d693b107f07bc9ab094bfaa","unresolved":false,"context_lines":[{"line_number":115,"context_line":"        # think there are no valid images and deleting everything."},{"line_number":116,"context_line":"        # self.my_api_server.needs_database \u003d False"},{"line_number":117,"context_line":"        # self.start_with_retry(self.my_api_server,"},{"line_number":118,"context_line":"        #                       \u0027api_port\u0027, 3, **self.__dict__.copy())"},{"line_number":119,"context_line":"        self.start_server(run_staging_cleaner\u003dTrue)"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        # Poll to give it time to come up and do the work. Use the presence"}],"source_content_type":"text/x-python","patch_set":3,"id":"33d14e86_26c8e6de","line":118,"range":{"start_line":118,"start_character":9,"end_line":118,"end_character":32},"in_reply_to":"29ca9b8a_1930422f","updated":"2025-06-16 17:13:18.000000000","message":"Acknowledged","commit_id":"3a48741395a9a6c15002d6b491f5d033ea83066f"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2d17abf47fd8703780bcef3004574698e11e20fa","unresolved":true,"context_lines":[{"line_number":153,"context_line":"    This tests the multistore case."},{"line_number":154,"context_line":"    \"\"\""},{"line_number":155,"context_line":"    def setUp(self):"},{"line_number":156,"context_line":"        super(TestStagingCleanupMultistore, self).setUp()"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"class TestStagingCleanupSingleStore(functional.SynchronousAPIBase,"}],"source_content_type":"text/x-python","patch_set":3,"id":"29335a02_7e414601","line":156,"range":{"start_line":156,"start_character":14,"end_line":156,"end_character":42},"updated":"2025-06-16 15:53:40.000000000","message":"You could even remove this method, but it\u0027s also fine to keep it :)","commit_id":"3a48741395a9a6c15002d6b491f5d033ea83066f"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"b077f2dd8c573a9b6d693b107f07bc9ab094bfaa","unresolved":false,"context_lines":[{"line_number":153,"context_line":"    This tests the multistore case."},{"line_number":154,"context_line":"    \"\"\""},{"line_number":155,"context_line":"    def setUp(self):"},{"line_number":156,"context_line":"        super(TestStagingCleanupMultistore, self).setUp()"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"class TestStagingCleanupSingleStore(functional.SynchronousAPIBase,"}],"source_content_type":"text/x-python","patch_set":3,"id":"663e6fb1_7a24b3c3","line":156,"range":{"start_line":156,"start_character":14,"end_line":156,"end_character":42},"in_reply_to":"29335a02_7e414601","updated":"2025-06-16 17:13:18.000000000","message":"I think this needs here as the class does not have any other code, so instead of having pass (which I don\u0027t know allowed or not), better to have setup","commit_id":"3a48741395a9a6c15002d6b491f5d033ea83066f"}]}