)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"405bf05ab4fb720087f18f56c30f1eafbf5c27fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d9c7f4b8_e5410ae0","updated":"2025-11-03 15:21:01.000000000","message":"recheck unrelated error","commit_id":"ab6bfb7e38e4701d09498e638bccef44c68dcfc8"}],"neutron/tests/functional/agent/ovn/metadata/test_metadata_agent.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"58253a53d0c3edfbc93fdf1c80059bbe2974041d","unresolved":true,"context_lines":[{"line_number":709,"context_line":"            \u0027Chassis_Private\u0027, self.chassis_name,"},{"line_number":710,"context_line":"            (\u0027external_ids\u0027, external_ids)).execute(check_error\u003dTrue)"},{"line_number":711,"context_line":""},{"line_number":712,"context_line":"        cp_event \u003d ChassisPrivateUpdateEvent(self.chassis_name)"},{"line_number":713,"context_line":"        self.agent.sb_idl.idl.notify_handler.watch_event(cp_event)"},{"line_number":714,"context_line":"        self.agent._cleanup_previous_tags()"},{"line_number":715,"context_line":"        self.assertTrue(cp_event.wait())"}],"source_content_type":"text/x-python","patch_set":1,"id":"b38339ed_17f07a33","side":"PARENT","line":712,"updated":"2025-11-11 20:12:17.000000000","message":"I think the reason this was failing is because the event needs to be added and watched *before* calling sb_api.db_set, not because we need an active wait.","commit_id":"a8e636cc58f89f6f67de8348f5512d7063e399ef"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"e0c8864aaa548470a2b79597057a8f639dd30895","unresolved":false,"context_lines":[{"line_number":709,"context_line":"            \u0027Chassis_Private\u0027, self.chassis_name,"},{"line_number":710,"context_line":"            (\u0027external_ids\u0027, external_ids)).execute(check_error\u003dTrue)"},{"line_number":711,"context_line":""},{"line_number":712,"context_line":"        cp_event \u003d ChassisPrivateUpdateEvent(self.chassis_name)"},{"line_number":713,"context_line":"        self.agent.sb_idl.idl.notify_handler.watch_event(cp_event)"},{"line_number":714,"context_line":"        self.agent._cleanup_previous_tags()"},{"line_number":715,"context_line":"        self.assertTrue(cp_event.wait())"}],"source_content_type":"text/x-python","patch_set":1,"id":"ce985b0a_bacfa5ce","side":"PARENT","line":712,"in_reply_to":"3438e95d_462b5213","updated":"2025-11-11 20:35:43.000000000","message":"I can\u0027t read. set !\u003d get :p, But I still want to figure out why this is actually failing. If the WaitEvent is broken somehow, I really want to understand why and make sure that it isn\u0027t something that happens in real code.","commit_id":"a8e636cc58f89f6f67de8348f5512d7063e399ef"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"02bc1afddb33f55be40fc7395a491b8daf0f416c","unresolved":true,"context_lines":[{"line_number":709,"context_line":"            \u0027Chassis_Private\u0027, self.chassis_name,"},{"line_number":710,"context_line":"            (\u0027external_ids\u0027, external_ids)).execute(check_error\u003dTrue)"},{"line_number":711,"context_line":""},{"line_number":712,"context_line":"        cp_event \u003d ChassisPrivateUpdateEvent(self.chassis_name)"},{"line_number":713,"context_line":"        self.agent.sb_idl.idl.notify_handler.watch_event(cp_event)"},{"line_number":714,"context_line":"        self.agent._cleanup_previous_tags()"},{"line_number":715,"context_line":"        self.assertTrue(cp_event.wait())"}],"source_content_type":"text/x-python","patch_set":1,"id":"3438e95d_462b5213","side":"PARENT","line":712,"in_reply_to":"b38339ed_17f07a33","updated":"2025-11-11 20:23:00.000000000","message":"Or maybe that the condition needs to be made more specific than just \"an update and the name matches, which the db_set() will also match.","commit_id":"a8e636cc58f89f6f67de8348f5512d7063e399ef"}]}
