)]}'
{"tests/unit/test_v3.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"cddabe7d23b6dddcf2852c7bd11cad5740566332","unresolved":false,"context_lines":[{"line_number":4673,"context_line":"        j \u003d json.loads(_get_file(paused_job.jobdir.result_data_file))"},{"line_number":4674,"context_line":"        self.assertEqual(j[\"data\"][\"build_id\"], paused_job.uuid)"},{"line_number":4675,"context_line":""},{"line_number":4676,"context_line":"        self.executor_server.hold_jobs_in_build \u003d False"},{"line_number":4677,"context_line":"        # Stop the job worker to simulate an executor restart"},{"line_number":4678,"context_line":"        for job_worker in self.executor_server.job_workers.values():"},{"line_number":4679,"context_line":"            if job_worker.build_request.uuid \u003d\u003d paused_job.uuid:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1542cfab_5b05e54b","line":4676,"updated":"2021-09-16 23:22:46.000000000","message":"Fundamentally, if moving this line fixes a race condition, then we weren\u0027t actually settled on line 4667.  That\u0027s the thing we need to fix.\n\nDo you have logs for this failure?","commit_id":"6f393b0cfa0eaf22ee271231130d0bfa69d2ab70"},{"author":{"_account_id":27952,"name":"Felix Edel","email":"felix.edel@bmw.de","username":"felix.schmidt"},"change_message_id":"946951714d86108bce4ed12caef339c77ace265e","unresolved":false,"context_lines":[{"line_number":4673,"context_line":"        j \u003d json.loads(_get_file(paused_job.jobdir.result_data_file))"},{"line_number":4674,"context_line":"        self.assertEqual(j[\"data\"][\"build_id\"], paused_job.uuid)"},{"line_number":4675,"context_line":""},{"line_number":4676,"context_line":"        self.executor_server.hold_jobs_in_build \u003d False"},{"line_number":4677,"context_line":"        # Stop the job worker to simulate an executor restart"},{"line_number":4678,"context_line":"        for job_worker in self.executor_server.job_workers.values():"},{"line_number":4679,"context_line":"            if job_worker.build_request.uuid \u003d\u003d paused_job.uuid:"}],"source_content_type":"text/x-python","patch_set":2,"id":"ce7d4c01_204839c1","line":4676,"in_reply_to":"1542cfab_5b05e54b","updated":"2021-09-21 06:44:02.000000000","message":"I\u0027ve put this change on top of https://review.opendev.org/c/zuul/zuul/+/808792/7 to see if it\u0027s still necessary. Without this \"fix\" the test case is failing for the change above, e.g. https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4da/809175/5/check/zuul-tox-py38/4daf1f6/testr_results.html\n\nHowever, thinking about that now, it might also be related to the change above which adapts the waitUntilSettled() method. I will have a look at it.","commit_id":"6f393b0cfa0eaf22ee271231130d0bfa69d2ab70"}]}