)]}'
{"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"90edac8c6ba2a040570719bab5999d08cfc0e78f","unresolved":false,"context_lines":[{"line_number":2556,"context_line":"            if bridges_recreated:"},{"line_number":2557,"context_line":"                # In case when any bridge was \"re-created\", we need to ensure"},{"line_number":2558,"context_line":"                # that there is no any stale flows in bridges left"},{"line_number":2559,"context_line":"                need_clean_stale_flow \u003d True"},{"line_number":2560,"context_line":"            sync |\u003d bridges_recreated"},{"line_number":2561,"context_line":"            # Notify the plugin of tunnel IP"},{"line_number":2562,"context_line":"            if self.enable_tunneling and tunnel_sync:"}],"source_content_type":"text/x-python","patch_set":1,"id":"df33271e_c29b7270","line":2559,"range":{"start_line":2559,"start_character":16,"end_line":2559,"end_character":37},"updated":"2020-04-09 14:24:47.000000000","message":"nit: need_clean_stale_flow |\u003d bridges_recreated\n\njust for consistency with next line","commit_id":"7bab6f48e9d1a323ef5034e020c998d1f9b38cae"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ffd829fead46e32ecc8fcb4b01fb330afc2e4931","unresolved":false,"context_lines":[{"line_number":2556,"context_line":"            if bridges_recreated:"},{"line_number":2557,"context_line":"                # In case when any bridge was \"re-created\", we need to ensure"},{"line_number":2558,"context_line":"                # that there is no any stale flows in bridges left"},{"line_number":2559,"context_line":"                need_clean_stale_flow \u003d True"},{"line_number":2560,"context_line":"            sync |\u003d bridges_recreated"},{"line_number":2561,"context_line":"            # Notify the plugin of tunnel IP"},{"line_number":2562,"context_line":"            if self.enable_tunneling and tunnel_sync:"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff570b3c_11cff07f","line":2559,"updated":"2020-05-11 10:41:31.000000000","message":"In case we need to clean the stale flows, we execute [1], but this is going to happen only when we have a port change, a restart or we need to refresh the FW [2]. If there are no changes, this cleanup won\u0027t be executed.\n\nCan I recommend to take this cleanup out of [2] branch?\n\n\n[1]https://github.com/openstack/neutron/blob/2ac52607c266e593700be0784ebadc77789070ff/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L2613\n[2]https://github.com/openstack/neutron/blob/2ac52607c266e593700be0784ebadc77789070ff/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L2603-L2605","commit_id":"b40c6db92418653c99c3d0f3b0365bc5a619497f"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"4da865f43bcc9198e7cee747a77df68e0088684e","unresolved":false,"context_lines":[{"line_number":2556,"context_line":"            if bridges_recreated:"},{"line_number":2557,"context_line":"                # In case when any bridge was \"re-created\", we need to ensure"},{"line_number":2558,"context_line":"                # that there is no any stale flows in bridges left"},{"line_number":2559,"context_line":"                need_clean_stale_flow \u003d True"},{"line_number":2560,"context_line":"            sync |\u003d bridges_recreated"},{"line_number":2561,"context_line":"            # Notify the plugin of tunnel IP"},{"line_number":2562,"context_line":"            if self.enable_tunneling and tunnel_sync:"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff570b3c_0b465450","line":2559,"in_reply_to":"ff570b3c_11cff07f","updated":"2020-05-13 21:10:56.000000000","message":"Done","commit_id":"b40c6db92418653c99c3d0f3b0365bc5a619497f"}]}