)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"afe9d78919110aa283fe9c3ab2b49ff3a9ece04d","unresolved":true,"context_lines":[{"line_number":12,"context_line":"different OF rules in br-int."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The problem is that such conntrack entries were cleaned for each of the"},{"line_number":15,"context_line":"ports for which filters were updated, even with defered apply on. In"},{"line_number":16,"context_line":"such case new OpenFlow rules are applied in defered way after all ports"},{"line_number":17,"context_line":"are processed by the agent."},{"line_number":18,"context_line":"That could lead to the situation where conntrack entries to mark some"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2b130578_068d3c6d","line":15,"range":{"start_line":15,"start_character":48,"end_line":15,"end_character":55},"updated":"2026-06-01 11:22:48.000000000","message":"deferred","commit_id":"c3f17c34a812ee161b42bfb5222462a655324c37"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ff885647f5e669e958a8fe3bc135ecf4c66120b0","unresolved":false,"context_lines":[{"line_number":12,"context_line":"different OF rules in br-int."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The problem is that such conntrack entries were cleaned for each of the"},{"line_number":15,"context_line":"ports for which filters were updated, even with defered apply on. In"},{"line_number":16,"context_line":"such case new OpenFlow rules are applied in defered way after all ports"},{"line_number":17,"context_line":"are processed by the agent."},{"line_number":18,"context_line":"That could lead to the situation where conntrack entries to mark some"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6e1ce42f_8841d245","line":15,"range":{"start_line":15,"start_character":48,"end_line":15,"end_character":55},"in_reply_to":"2b130578_068d3c6d","updated":"2026-06-01 12:27:27.000000000","message":"Done","commit_id":"c3f17c34a812ee161b42bfb5222462a655324c37"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"afe9d78919110aa283fe9c3ab2b49ff3a9ece04d","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The problem is that such conntrack entries were cleaned for each of the"},{"line_number":15,"context_line":"ports for which filters were updated, even with defered apply on. In"},{"line_number":16,"context_line":"such case new OpenFlow rules are applied in defered way after all ports"},{"line_number":17,"context_line":"are processed by the agent."},{"line_number":18,"context_line":"That could lead to the situation where conntrack entries to mark some"},{"line_number":19,"context_line":"kind of connection as invalid was deleted by the fw driver, but new OF"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"aa20ae3a_74250eaa","line":16,"range":{"start_line":16,"start_character":44,"end_line":16,"end_character":51},"updated":"2026-06-01 11:22:48.000000000","message":"deferred","commit_id":"c3f17c34a812ee161b42bfb5222462a655324c37"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ff885647f5e669e958a8fe3bc135ecf4c66120b0","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The problem is that such conntrack entries were cleaned for each of the"},{"line_number":15,"context_line":"ports for which filters were updated, even with defered apply on. In"},{"line_number":16,"context_line":"such case new OpenFlow rules are applied in defered way after all ports"},{"line_number":17,"context_line":"are processed by the agent."},{"line_number":18,"context_line":"That could lead to the situation where conntrack entries to mark some"},{"line_number":19,"context_line":"kind of connection as invalid was deleted by the fw driver, but new OF"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c28eddd8_f4475288","line":16,"range":{"start_line":16,"start_character":44,"end_line":16,"end_character":51},"in_reply_to":"aa20ae3a_74250eaa","updated":"2026-06-01 12:27:27.000000000","message":"Done","commit_id":"c3f17c34a812ee161b42bfb5222462a655324c37"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4e8a6ed071c32180ef162a8995bf5ca11cacda10","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4e1c905e_e104c68c","updated":"2026-06-01 15:19:53.000000000","message":"Yeah, something like this.","commit_id":"8fde7d461a3ee44be266946dd74d3716df714818"}],"neutron/agent/linux/openvswitch_firewall/firewall.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"afe9d78919110aa283fe9c3ab2b49ff3a9ece04d","unresolved":true,"context_lines":[{"line_number":718,"context_line":""},{"line_number":719,"context_line":"    def _flush_pending_invalid_conntrack_cleanup(self):"},{"line_number":720,"context_line":"        for port, of_port in self._ports_pending_invalid_ct_cleanup.values():"},{"line_number":721,"context_line":"            self._delete_invalid_conntrack_entries_for_port(port, of_port)"},{"line_number":722,"context_line":"        self._ports_pending_invalid_ct_cleanup.clear()"},{"line_number":723,"context_line":""},{"line_number":724,"context_line":"    def get_ofport(self, port):"}],"source_content_type":"text/x-python","patch_set":1,"id":"c341ab8a_3c0cd4e8","line":721,"range":{"start_line":721,"start_character":17,"end_line":721,"end_character":59},"updated":"2026-06-01 11:22:48.000000000","message":"Maybe we can pass a port list to `_delete_invalid_conntrack_entries_for_port`. The `self.ipconntrack.delete_conntrack_state_by_remote_ips` accepts a `device_info_list`. Actually in `_delete_invalid_conntrack_entries_for_port` we are passing a single item list to this second method.","commit_id":"c3f17c34a812ee161b42bfb5222462a655324c37"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ff885647f5e669e958a8fe3bc135ecf4c66120b0","unresolved":false,"context_lines":[{"line_number":718,"context_line":""},{"line_number":719,"context_line":"    def _flush_pending_invalid_conntrack_cleanup(self):"},{"line_number":720,"context_line":"        for port, of_port in self._ports_pending_invalid_ct_cleanup.values():"},{"line_number":721,"context_line":"            self._delete_invalid_conntrack_entries_for_port(port, of_port)"},{"line_number":722,"context_line":"        self._ports_pending_invalid_ct_cleanup.clear()"},{"line_number":723,"context_line":""},{"line_number":724,"context_line":"    def get_ofport(self, port):"}],"source_content_type":"text/x-python","patch_set":1,"id":"47a2c49f_e1d6daf5","line":721,"range":{"start_line":721,"start_character":17,"end_line":721,"end_character":59},"in_reply_to":"c341ab8a_3c0cd4e8","updated":"2026-06-01 12:27:27.000000000","message":"Done","commit_id":"c3f17c34a812ee161b42bfb5222462a655324c37"}]}