)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a693905dca5a31927fdc483ebc474c3facb1c8e2","unresolved":true,"context_lines":[{"line_number":7,"context_line":"SubclassSignatureTestCase to use NoDBTestCase as base"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We have a list of fixtures included in the test.TestCase base class"},{"line_number":10,"context_line":"that prevents global data and tread leaking across test cases within"},{"line_number":11,"context_line":"the same process. The SubclassSignatureTestCase did not use our base"},{"line_number":12,"context_line":"class but it initializes a partial libvirt driver class that will soon"},{"line_number":13,"context_line":"use a ThreadPoolExecutor in native threading mode. So we need the leak"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f1faca0_2fe11503","line":10,"range":{"start_line":10,"start_character":30,"end_line":10,"end_character":35},"updated":"2026-01-27 14:04:20.000000000","message":"thread","commit_id":"f394b424269534b81434628760a0b99a5db7bc07"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"73aa4ced7772d81e593d3e832e2a7e92ac062635","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4f50a2eb_9acba309","updated":"2026-01-27 15:15:14.000000000","message":"easy one","commit_id":"f394b424269534b81434628760a0b99a5db7bc07"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"86c94f80d6c9ac711d9ebd1957da54aabcdbefd4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"92b85ac0_dbfeeb58","updated":"2026-02-04 20:31:37.000000000","message":"i dont actully recall wehre we use SubclassSignatureTestCase i assume to validate teh virt driver interface but yes inherting form the no db testcase makes sense.","commit_id":"c89e54cedc2a889b8e967be5b06a60d09ee1b81d"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c000d53d6945e77b2ef465a046943986b889adf6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"013779c8_9c277723","updated":"2026-02-02 01:20:06.000000000","message":"lgtm","commit_id":"c89e54cedc2a889b8e967be5b06a60d09ee1b81d"}],"nova/test.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a693905dca5a31927fdc483ebc474c3facb1c8e2","unresolved":false,"context_lines":[{"line_number":759,"context_line":"        raise NotImplementedError()"},{"line_number":760,"context_line":""},{"line_number":761,"context_line":"    def setUp(self):"},{"line_number":762,"context_line":"        self.useFixture(nova_fixtures.ConfFixture(CONF))"},{"line_number":763,"context_line":"        self.base \u003d self._get_base_class()"},{"line_number":764,"context_line":""},{"line_number":765,"context_line":"        super(SubclassSignatureTestCase, self).setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"a58d010d_7ca6b87e","side":"PARENT","line":762,"updated":"2026-01-27 14:04:20.000000000","message":"note that conffixture is included into the new base class be default.","commit_id":"ed8462b25f2e4c4b32e27eba98dec621bb972a65"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a693905dca5a31927fdc483ebc474c3facb1c8e2","unresolved":false,"context_lines":[{"line_number":841,"context_line":"        self.addCleanup(timeutils.clear_time_override)"},{"line_number":842,"context_line":""},{"line_number":843,"context_line":""},{"line_number":844,"context_line":"class NoDBTestCase(TestCase):"},{"line_number":845,"context_line":"    \"\"\"`NoDBTestCase` differs from TestCase in that DB access is not supported."},{"line_number":846,"context_line":"    This makes tests run significantly faster. If possible, all new tests"},{"line_number":847,"context_line":"    should derive from this class."}],"source_content_type":"text/x-python","patch_set":1,"id":"af7d139a_ba0b5dc1","side":"PARENT","line":844,"updated":"2026-01-27 14:04:20.000000000","message":"I needed to move the class def earlier in the file to be able to use it earlier","commit_id":"ed8462b25f2e4c4b32e27eba98dec621bb972a65"}]}