)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"336aba0e590101620afd31d512e0dd10dd548392","unresolved":true,"context_lines":[{"line_number":7,"context_line":"ovn-evpn: Fix race in EVPN FSM functional test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The `fsm_advance` mock calls the real `fsm.advance` via `wrap`. The real"},{"line_number":10,"context_line":"instance will run in a separate thread then the mock object. By design,"},{"line_number":11,"context_line":"mock.called will return True as soon as the function was called. That"},{"line_number":12,"context_line":"does not mean that the real advance has actually finished. The real advance"},{"line_number":13,"context_line":"function sets evpn instance attributes in its callback function."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6a11de06_06092c93","line":10,"range":{"start_line":10,"start_character":39,"end_line":10,"end_character":43},"updated":"2026-09-11 14:38:44.000000000","message":"nit: than","commit_id":"355683ea7da4037a5ecc2e79f0b2c0e19ce50382"}],"neutron/tests/functional/agent/ovn/extensions/evpn/test_events.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"336aba0e590101620afd31d512e0dd10dd548392","unresolved":true,"context_lines":[{"line_number":150,"context_line":"        self.fsm \u003d evpn_extension._evpn_fsm"},{"line_number":151,"context_line":"        self.fsm_advance \u003d mock.patch.object("},{"line_number":152,"context_line":"            self.fsm, \u0027advance\u0027, wraps\u003dself.fsm.advance).start()"},{"line_number":153,"context_line":"        self.fsm_advance.reset_mock()"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    def _start_evpn_agent(self):"},{"line_number":156,"context_line":"        conf \u003d self.useFixture(fixture_config.Config()).conf"}],"source_content_type":"text/x-python","patch_set":1,"id":"c10d3f72_bf6d5df4","line":153,"updated":"2026-09-11 14:38:44.000000000","message":"The object was just mocked, is it really needed?","commit_id":"355683ea7da4037a5ecc2e79f0b2c0e19ce50382"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"a2c076e50217ddd952bf3d38b85c0b453a95eccb","unresolved":true,"context_lines":[{"line_number":150,"context_line":"        self.fsm \u003d evpn_extension._evpn_fsm"},{"line_number":151,"context_line":"        self.fsm_advance \u003d mock.patch.object("},{"line_number":152,"context_line":"            self.fsm, \u0027advance\u0027, wraps\u003dself.fsm.advance).start()"},{"line_number":153,"context_line":"        self.fsm_advance.reset_mock()"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    def _start_evpn_agent(self):"},{"line_number":156,"context_line":"        conf \u003d self.useFixture(fixture_config.Config()).conf"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a46bfd9_b95c3edf","line":153,"in_reply_to":"c10d3f72_bf6d5df4","updated":"2026-09-11 16:32:22.000000000","message":"I was also wondering the same. I\u0027ll wait for @twilson@redhat.com review to see if we can possibly remove it.","commit_id":"355683ea7da4037a5ecc2e79f0b2c0e19ce50382"}]}