[ovs-dev] [PATCH ovn 01/12] tests: Improve "dhcpv4 : 1 HV, 2 LS, 2 LSPs/LS".

Numan Siddique numans at ovn.org
Wed Nov 4 10:06:20 UTC 2020


On Wed, Nov 4, 2020 at 12:33 PM Ben Pfaff <blp at ovn.org> wrote:
>
> This change makes the test much more debuggable:
>
> - Make it easier to find the particular failing test.
>
> - Use correct checksums so that the packets can be compared for
>   equality instead of omitting the checksum field.
>
> - Factor out packet comparison and produce better error messages on
>   failure.
>
> Signed-off-by: Ben Pfaff <blp at ovn.org>

Thanks for the series. Patches 1 to 6 of this series were
straightforward. So I applied to the main branch.

Thanks
Numan

> ---
>  tests/network-functions.at |  18 ++++
>  tests/ovn.at               | 184 ++++++++++++-------------------------
>  2 files changed, 75 insertions(+), 127 deletions(-)
>
> diff --git a/tests/network-functions.at b/tests/network-functions.at
> index a149e9da4c58..c583bc31e881 100644
> --- a/tests/network-functions.at
> +++ b/tests/network-functions.at
> @@ -52,6 +52,24 @@ test_csum 45000076000000004001c3d90a000003aca80003 0000
>  AT_CLEANUP
>
>  OVS_START_SHELL_HELPERS
> +# ip4_csum_inplace IP4_HEADER
> +#
> +# Outputs IP4_HEADER with the checksum filled in properly.
> +# The checksum must initially be 0000.  IP4_HEADER must be
> +# 40 hex digits.
> +ip4_csum_inplace() {
> +    local csum=$(ip_csum $1)
> +    echo "$1" | sed "s/^\(....................\)..../\1$csum/"
> +}
> +OVS_END_SHELL_HELPERS
> +
> +AT_SETUP([ip4_csum_inplace])
> +AT_CHECK([ip4_csum_inplace 450000730000400040110000c0a80001c0a800c7], [0],
> +    [45000073000040004011b861c0a80001c0a800c7
> +])
> +AT_CLEANUP
> +
> +OVS_START_SHELL_HELPERS
>  # ip6_pseudoheader IP6_HEADER NEXT_HEADER PAYLOAD_LEN
>  #
>  # where:
> diff --git a/tests/ovn.at b/tests/ovn.at
> index 4be484051966..5666058d99ca 100644
> --- a/tests/ovn.at
> +++ b/tests/ovn.at
> @@ -5294,7 +5294,12 @@ sleep 2
>  as hv1 ovs-vsctl show
>
>  # This shell function sends a DHCP request packet
> -# test_dhcp INPORT SRC_MAC DHCP_TYPE BROADCAST CIADDR OFFER_IP REQUEST_IP ETH_BOOT USE_IP ...
> +#
> +# The first argument is just the number of calls to this function so
> +# far (1, 2, ...).  This is redundant, but it makes it easier to find
> +# the failures by searching for the number.
> +#
> +# test_dhcp PACKET_NUM INPORT SRC_MAC DHCP_TYPE BROADCAST CIADDR OFFER_IP REQUEST_IP ETH_BOOT USE_IP ...
>  packet_num=0
>  test_dhcp() {
>      local expect_resume=:
> @@ -5308,6 +5313,10 @@ test_dhcp() {
>          esac
>      done
>
> +    packet_num=$(expr $packet_num + 1)
> +    AT_FAIL_IF([test $packet_num != $1])
> +    shift
> +
>      local inport=$1 src_mac=$2 dhcp_type=$3 broadcast=$4 ciaddr=$5 offer_ip=$6 request_ip=$7 eth_boot=$8 use_ip=$9
>      shift; shift; shift; shift; shift; shift; shift; shift; shift;
>
> @@ -5319,7 +5328,6 @@ test_dhcp() {
>          src_ip=`ip_to_hex 0 0 0 0`
>          dst_ip=`ip_to_hex 255 255 255 255`
>      fi
> -    packet_num=$(expr $packet_num + 1)
>
>      AS_BOX([dhcp test packet $packet_num])
>
> @@ -5415,7 +5423,9 @@ test_dhcp() {
>          ip_len=$(printf "%x" $ip_len)
>          udp_len=$(printf "%x" $udp_len)
>          # $ip_len var will be in 3 digits i.e 134. So adding a '0' before $ip_len
> -        local reply=${src_mac}${srv_mac}080045100${ip_len}000000008011XXXX${srv_ip}${reply_dst_ip}
> +        local reply=${src_mac}${srv_mac}0800
> +        local ip_header=45100${ip_len}0000000080110000${srv_ip}${reply_dst_ip}
> +        reply=${reply}$(ip4_csum_inplace $ip_header)
>          # udp header and dhcp header.
>          # $udp_len var will be in 3 digits. So adding a '0' before $udp_len
>          reply=${reply}004300440${udp_len}0000020106006359aa760000${flags}${ciaddr}
> @@ -5460,6 +5470,16 @@ test_dhcp() {
>      fi
>  }
>
> +compare_dhcp_packets() {
> +    received=$($PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif$1-tx.pcap)
> +    expected=$(cat $1.expected)
> +
> +    if test "$received" != "$expected"; then
> +        AT_CHECK_UNQUOTED([echo "$received"; tcpdump_hex "$received"], [0],
> +            [$(echo "$expected"; tcpdump_hex "$expected")])
> +    fi
> +}
> +
>  reset_pcap_file() {
>      local iface=$1
>      local pcap_file=$2
> @@ -5481,14 +5501,8 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=0
>  expected_dhcp_opts=330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 1 f00000000001 01 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif1-tx.pcap > 1.packets
> -cat 1.expected | cut -c -48 > expout
> -AT_CHECK([cat 1.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 1.expected | cut -c 53- > expout
> -AT_CHECK([cat 1.packets | cut -c 53-], [0], [expout])
> +test_dhcp 1 1 f00000000001 01 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
> +compare_dhcp_packets 1
>
>  # ----------------------------------------------------------------------
>
> @@ -5506,14 +5520,8 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=$offer_ip
>  expected_dhcp_opts=330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 2 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5529,14 +5537,8 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=`ip_to_hex 10 0 0 7`
>  expected_dhcp_opts=""
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 06 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 3 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 06 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>  reset_pcap_file hv1-vif1 hv1/vif1
> @@ -5551,7 +5553,7 @@ rm -f 2.expected
>  ciaddr=`ip_to_hex 0 0 0 0`
>  offer_ip=0
>  request_ip=0
> -test_dhcp 2 f00000000002 09 0 $ciaddr $offer_ip $request_ip 0 0 1 1
> +test_dhcp 4 2 f00000000002 09 0 $ciaddr $offer_ip $request_ip 0 0 1 1
>
>  # NXT_RESUMEs should be 4.
>  # vif1-tx.pcap should have received the DHCPv4 (invalid) request packet
> @@ -5574,14 +5576,8 @@ request_ip=0
>  src_ip=$offer_ip
>  dst_ip=$server_ip
>  expected_dhcp_opts=330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 5 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5599,14 +5595,8 @@ request_ip=0
>  src_ip=$offer_ip
>  dst_ip=`ip_to_hex 255 255 255 255`
>  expected_dhcp_opts=330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 6 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5624,14 +5614,8 @@ request_ip=0
>  src_ip=$offer_ip
>  dst_ip=`ip_to_hex 255 255 255 255`
>  expected_dhcp_opts=""
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 06 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 7 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 06 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5649,14 +5633,8 @@ request_ip=0
>  src_ip=$offer_ip
>  dst_ip=`ip_to_hex 255 255 255 255`
>  expected_dhcp_opts=""
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 06 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 8 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 06 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5670,7 +5648,7 @@ rm -f 2.expected
>  ciaddr=`ip_to_hex 0 0 0 0`
>  src_ip=`ip_to_hex 10 0 0 6`
>  dst_ip=`ip_to_hex 10 0 0 4`
> -test_dhcp --no-resume 2 f00000000002 03 0 $ciaddr 0 0 0 1 $src_ip $dst_ip 1
> +test_dhcp --no-resume 9 2 f00000000002 03 0 $ciaddr 0 0 0 1 $src_ip $dst_ip 1
>
>  # vif1-tx.pcap should have received the DHCPv4 request packet
>  OVN_CHECK_PACKETS([hv1/vif1-tx.pcap], [1.expected])
> @@ -5688,14 +5666,8 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=0
>  expected_dhcp_opts=330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 1 f00000000001 01 1 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif1-tx.pcap > 1.packets
> -cat 1.expected | cut -c -48 > expout
> -AT_CHECK([cat 1.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 1.expected | cut -c 53- > expout
> -AT_CHECK([cat 1.packets | cut -c 53-], [0], [expout])
> +test_dhcp 10 1 f00000000001 01 1 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
> +compare_dhcp_packets 1
>
>  # ----------------------------------------------------------------------
>
> @@ -5710,7 +5682,7 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 10 0 0 6`
>  request_ip=0
>  expected_dhcp_opts=0
> -test_dhcp 2 f00000000002 07 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001
> +test_dhcp 11 2 f00000000002 07 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001
>
>  # There is no reply for this. Check for the INFO log in ovn-controller.log
>  AT_CHECK([test 1 = $(cat hv1/ovn-controller.log | \
> @@ -5736,14 +5708,8 @@ dst_ip=$server_ip
>  # In the expected_dhcp_opts we should not see 330400000e10 which is
>  # dhcp lease time option and 0104ffffff00 which is subnet mask option.
>  expected_dhcp_opts=03040a00000136040a000001
> -test_dhcp 2 f00000000002 08 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 12 2 f00000000002 08 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # Now add the dhcp option T1 to the dhcp options.
>  check ovn-nbctl --wait=hv set dhcp_options ${d1} options:T1=4000
> @@ -5767,14 +5733,8 @@ dst_ip=$server_ip
>  # In the expected_dhcp_opts we should not see 330400000e10 which is
>  # dhcp lease time option.
>  expected_dhcp_opts=3a0400000fa0330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 13 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5793,14 +5753,8 @@ dst_ip=$server_ip
>  # In the expected_dhcp_opts we should not see 330400000e10 which is
>  # dhcp lease time option and 0104ffffff00 which is subnet mask option.
>  expected_dhcp_opts=03040a00000136040a000001
> -test_dhcp 2 f00000000002 08 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 14 2 f00000000002 08 0 $ciaddr $offer_ip $request_ip 0 1 $src_ip $dst_ip ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5825,14 +5779,8 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=$offer_ip
>  expected_dhcp_opts=330400000e100104ffffff0003040a00000136040a00000142040a0a0a0a
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 15 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5857,14 +5805,8 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=$offer_ip
>  expected_dhcp_opts=330400000e100104ffffff0003040a00000136040a0000014210746573745f746674705f736572766572
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 16 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5889,14 +5831,8 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=$offer_ip
>  expected_dhcp_opts=771305746573743103636f6d00057465737432c006330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif2-tx.pcap > 2.packets
> -cat 2.expected | cut -c -48 > expout
> -AT_CHECK([cat 2.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 2.expected | cut -c 53- > expout
> -AT_CHECK([cat 2.packets | cut -c 53-], [0], [expout])
> +test_dhcp 17 2 f00000000002 03 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 05 $expected_dhcp_opts
> +compare_dhcp_packets 2
>
>  # ----------------------------------------------------------------------
>
> @@ -5906,7 +5842,7 @@ server_ip=`ip_to_hex 10 0 0 1`
>  ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=0
>  expected_dhcp_opts=""
> -test_dhcp 1 f00000000001 04 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
> +test_dhcp 18 1 f00000000001 04 0 $ciaddr $offer_ip $request_ip 0 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
>  AT_CHECK([fgrep -iq 'DHCPDECLINE from f0:00:00:00:00:01, 10.0.0.4 duplicated' hv1/ovn-controller.log], [0], [])
>
>  # Send Etherboot.
> @@ -5933,14 +5869,8 @@ ciaddr=`ip_to_hex 0 0 0 0`
>  request_ip=0
>  boofile=4308626f6f7466696c65
>  expected_dhcp_opts=${boofile}330400000e100104ffffff0003040a00000136040a000001
> -test_dhcp 1 f00000000001 01 0 $ciaddr $offer_ip $request_ip 1 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
> -
> -$PYTHON "$ovs_srcdir/utilities/ovs-pcap.in" hv1/vif1-tx.pcap > 1.packets
> -cat 1.expected | cut -c -48 > expout
> -AT_CHECK([cat 1.packets | cut -c -48], [0], [expout])
> -# Skipping the IPv4 checksum.
> -cat 1.expected | cut -c 53- > expout
> -AT_CHECK([cat 1.packets | cut -c 53-], [0], [expout])
> +test_dhcp 19 1 f00000000001 01 0 $ciaddr $offer_ip $request_ip 1 0 ff1000000001 $server_ip 02 $expected_dhcp_opts
> +compare_dhcp_packets 1
>
>  OVN_CLEANUP([hv1])
>
> --
> 2.26.2
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>


More information about the dev mailing list