On Mon, Sep 28, 2026 at 8:04 PM Jacob Tanenbaum via dev < [email protected]> wrote:
> 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 5b6ba3731..c2dcf667f 100644 > --- a/tests/system-ovn.at > +++ b/tests/system-ovn.at > @@ -7987,7 +7987,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 > ]) > @@ -7998,7 +7998,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 > > Thank you Jacob, applied to main. Regards, Ales _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
