forked from ovn-org/ovn
-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
ovn-controller: Fix the missing ct zone entries for container ports.
After the commit in the Fixes tag, ovn-controller was not creating ct zone entries for the container ports in the integration bridge's external_ids column. Because of this, when a container port sends a traffic to load balancer VIP, zone id is not used (because REG13 is not set). But the reverse traffic doesn't go through the ct_lb action for undnat, but instead go to the conntrack via the ct_commit() OVN action and the packet gets dropped. This happens if an ACL with allow-related action which matches in the egress pipeline of the logical switch. This patch fixes this regression and the tests make sure the the ct zone entries are created for the container ports. Fixes: 6c8b9a1("ovn-controller: Store the local port bindings in the runtime data I-P state.") Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1857865 Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1858191 Acked-by: Dumitru Ceara <dceara@redhat.com> Signed-off-by: Numan Siddique <numans@ovn.org>
- Loading branch information
1 parent
f8a8169
commit 925632e
Showing
3 changed files
with
159 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters