)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9617c332624e62e0aef9c076adcaf72a448fa4ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dd4b138f_cd834f40","updated":"2022-06-28 12:43:33.000000000","message":"+.5\n\ni think this makes sense just wondering if you ran this in a loop or otherwise did somethign to test the stablity of the sleep(1)","commit_id":"b00f4aa555fd935da4903f6cc148ece6a1ecf2b7"}],"nova/tests/functional/regressions/test_bug_1838392.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9617c332624e62e0aef9c076adcaf72a448fa4ee","unresolved":true,"context_lines":[{"line_number":27,"context_line":"    \"\"\""},{"line_number":28,"context_line":"    microversion \u003d \u0027latest\u0027"},{"line_number":29,"context_line":"    ADMIN_API \u003d True"},{"line_number":30,"context_line":"    # we need real parallel API behavior for this test"},{"line_number":31,"context_line":"    CAST_AS_CALL \u003d False"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def setUp(self):"},{"line_number":34,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9317d33e_2dfc2376","line":31,"range":{"start_line":30,"start_character":1,"end_line":31,"end_character":24},"updated":"2022-06-28 12:43:33.000000000","message":"do we if so how are you forcing the race in the order we need","commit_id":"b00f4aa555fd935da4903f6cc148ece6a1ecf2b7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9617c332624e62e0aef9c076adcaf72a448fa4ee","unresolved":true,"context_lines":[{"line_number":50,"context_line":"        server \u003d self.api.post_server({\u0027server\u0027: body})"},{"line_number":51,"context_line":"        self._wait_for_state_change(server, \"ACTIVE\")"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        from nova.virt.libvirt import driver"},{"line_number":54,"context_line":"        orig_cleanup \u003d driver.LibvirtDriver._cleanup"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        calls \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"7526e2d3_304cb2c7","line":53,"range":{"start_line":53,"start_character":8,"end_line":53,"end_character":44},"updated":"2022-06-28 12:43:33.000000000","message":"why are you doing this here?","commit_id":"b00f4aa555fd935da4903f6cc148ece6a1ecf2b7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9617c332624e62e0aef9c076adcaf72a448fa4ee","unresolved":true,"context_lines":[{"line_number":64,"context_line":"            if not calls:"},{"line_number":65,"context_line":"                calls.append(True)"},{"line_number":66,"context_line":"                self.api.delete_server(server[\u0027id\u0027])"},{"line_number":67,"context_line":"                # Let the delete progress. We cannot wait for the delete to"},{"line_number":68,"context_line":"                # finish as the original bug is fixed with a lock making"},{"line_number":69,"context_line":"                # this delete call waiting for the reboot to finish which would"},{"line_number":70,"context_line":"                # be a deadlock here."},{"line_number":71,"context_line":"                time.sleep(1)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"            return orig_cleanup(*args, **kwargs)"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9fbd5b92_5bf7045f","line":71,"range":{"start_line":67,"start_character":15,"end_line":71,"end_character":29},"updated":"2022-06-28 12:43:33.000000000","message":"hum i guess this will force a context switch to the pending rpc call.\ncoould we hook the lock  somehow instead?\n\nim asking because im not sure how reliable this will be for forcing the ordering.","commit_id":"b00f4aa555fd935da4903f6cc148ece6a1ecf2b7"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8e45e27c793ef1604c2e0a7aadd8dd526d21ca25","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2bf12adb_9d3b8925","line":90,"updated":"2022-06-28 12:49:04.000000000","message":"ah this is the pep8 failure two lines at the end of the file","commit_id":"b00f4aa555fd935da4903f6cc148ece6a1ecf2b7"}]}