git sha: 4dbd0b3e9921c9c7c1249b1d16e7ec71e5462d61
Author: Eelco Chaudron <[email protected]>
Subject: tests: Fix NSH decap header test for real Ethernet devices.

This commit modifies a test pattern to allow for optional padding bytes
at the end of Ethernet frames to accommodate the minimum 64-byte frame
requirement on real hardware.

> diff --git a/tests/system-traffic.at b/tests/system-traffic.at
> index 58a46af0a..c7f082fb1 100644
> --- a/tests/system-traffic.at
> +++ b/tests/system-traffic.at
> @@ -9893,7 +9893,7 @@ NS_CHECK_EXEC([at_ns0], [$PYTHON3 $srcdir/sendpkt.py p0 
> \
>  
>  dnl Check the expected de-capsulated TCP packet on the egress interface
>  OVS_WAIT_UNTIL([ovs-pcap p1.pcap | grep -q \
> -    "^$(ovs-ofctl compose-packet --bare 'TCP_SYN_PKT')\$"])
> +    "^$(ovs-ofctl compose-packet --bare 'TCP_SYN_PKT')0*\$"])

Would this regex pattern correctly handle cases where no padding is
present? The `0*` pattern allows zero or more zero characters, but does
this account for both the absence of padding bytes and the presence of
actual padding bytes that might be rendered differently by ovs-pcap?

>  
>  OVS_TRAFFIC_VSWITCHD_STOP
>  AT_CLEANUP
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to