Currently, the Copp test changes the log settings for vconn to check for the incoming health check packet. That isn't wrong but it is generic and would apply to all rconns. Switching to pinctrl:file:dbg is the right module the packet-in is processed by pinctrl - process_packet_in() which dispatches it to pinctrl_handle_svc_check(), pinctrl even has its own dedicated debug print for these packets.
Signed-off-by: Jacob Tanenbaum <[email protected]> --- tests/system-ovn.at | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/system-ovn.at b/tests/system-ovn.at index d1c35199e..aef6e3f29 100644 --- a/tests/system-ovn.at +++ b/tests/system-ovn.at @@ -8110,7 +8110,7 @@ check ovn-nbctl set nb_global . options:svc_monitor_mac="33:33:33:33:33:33" check ovn-nbctl meter-add svc-meter drop 1 pktps 0 check ovn-nbctl copp-add copp4 svc-monitor svc-meter check ovn-nbctl --wait=hv ls-copp-add copp4 sw0 -check ovn-appctl -t ovn-controller vlog/set vconn:dbg +check ovn-appctl -t ovn-controller vlog/set pinctrl:file:dbg AT_CHECK([ovn-nbctl copp-list copp4], [0], [dnl svc-monitor: svc-meter ]) @@ -8121,7 +8121,7 @@ p = Ether(dst="33:33:33:33:33:33", src="f0:00:00:01:02:03") /\ sendp(p, iface='sw01', loop=0, verbose=0, count=20) EOF -OVS_WAIT_UNTIL([test "1" = "$(grep -c "dl_dst=33:33:33:33:33:33" ovn-controller.log)"]) +OVS_WAIT_UNTIL([test "1" = "$(grep -c "dst-mac=33:33:33:33:33:33" ovn-controller.log)"]) OVN_CLEANUP_CONTROLLER([hv1]) -- 2.55.0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
