On Wed, Sep 23, 2026 at 3:24 PM Jacob Tanenbaum <[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]>
>

strange issue with an unrelated test case. Attempt the tests again

Recheck-request: github-robot-_Build_and_Test



> ---
>  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

Reply via email to