)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"db97cf05de89a5f2b5ccfc0433b59962639dac33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e766569c_e23cc0ab","updated":"2024-09-16 19:07:55.000000000","message":"Hi - can you file a bug for this?","commit_id":"1b81f28e909dc3a6a123c1e2b5d39f13050ddd58"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0fcf29b6f30cda8ce21447209781f3f15b493dd5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3799789f_58add734","updated":"2024-12-17 00:49:50.000000000","message":"I think this fix will help in the failure we keep seeing in the gate. The other alternative is to revert this other change:\n\nhttps://review.opendev.org/c/openstack/neutron/+/937758","commit_id":"4531efd253b9d0bdca83e539ce5b4e9f319009a4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"5679b15cfed7738f3bacea603972c456161b2a3b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0f758763_3656f44b","updated":"2024-12-17 11:33:13.000000000","message":"recheck ovs rally","commit_id":"4531efd253b9d0bdca83e539ce5b4e9f319009a4"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"e011fbd9f5ca4da2bb6bacb157716d30dba6ac1e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cd6f2b78_6830754d","in_reply_to":"3799789f_58add734","updated":"2024-12-17 11:54:24.000000000","message":"Ye i see this should help in the issues we seeing in functional tests. But considering https://github.com/openstack/neutron/blob/master/neutron/agent/l3/ha_router.py#L188-L189 the original state will always be \"backup\" so we shouldn\u0027t need the initial_stage_change method at all so good to get with the revert patch and always add a delay when state is primary.","commit_id":"4531efd253b9d0bdca83e539ce5b4e9f319009a4"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"7cd44e12fb1bd429d439edf49a43d7882db02bb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"45890dea_25fab36b","in_reply_to":"cd6f2b78_6830754d","updated":"2024-12-17 13:13:07.000000000","message":"The initial state in memory is populated by calling initial_state_change.\nIt\u0027s not taken the keepalive manager IIRC.","commit_id":"4531efd253b9d0bdca83e539ce5b4e9f319009a4"}],"neutron/agent/l3/ha.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"109b282046814a78ae6fcf688872d05dd2ff993e","unresolved":false,"context_lines":[{"line_number":94,"context_line":"        self._initial_state_change_per_router \u003d set()"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def initial_state_change(self, router_id):"},{"line_number":97,"context_line":"        initial_state \u003d router_id not in self._initial_state_change_per_router"},{"line_number":98,"context_line":"        if initial_state:"},{"line_number":99,"context_line":"            self._initial_state_change_per_router.add(router_id)"},{"line_number":100,"context_line":"        return initial_state"}],"source_content_type":"text/x-python","patch_set":3,"id":"7e03c9de_fd5da2f7","line":97,"updated":"2024-12-17 13:14:27.000000000","message":"this is a set, which is populated by calling this method.","commit_id":"4531efd253b9d0bdca83e539ce5b4e9f319009a4"}]}