-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
tests: Inject ARP replies for snoop tests on different port
The ARP replies injected into the underlay bridge 'br0' to trigger ARP snooping should be destined to the the bridges LOCAL port. So far the tests injected them on LOCAL port 'br0' itself, which didn't matter as OVS snooped on all ARP packets passing the bridge. This patch injects the ARP replies on a different port in preparation for an upcoming commit that will make OVS only snoop on ARP packets output to the LOCAL port. The clone() wrapper must be added to the generated datapath flows now as the traced packets would actually be transmitted through the tunnel port. Previously the underlay bridge dropped the packets as the learned egress port for the tunnel nexthop was the LOCAL port, which also served as virtual ingress port for the encapsulated traffic. The translation end result was an expensive way to say 'drop'. Signed-off-by: Zoltan Balogh <zoltan.balogh.eth@gmail.com> Co-authored-by: Jan Scheurich <jan.scheurich@ericsson.com> Signed-off-by: Jan Scheurich <jan.scheurich@ericsson.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
- Loading branch information
Showing
2 changed files
with
19 additions
and
19 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