Hi Ales, Thanks for the review and spotting that the "Reported-at" pointed towards the Epic instead of Upstream issue. Will fix in v2.
Thanks Xavier On Mon, Nov 24, 2025 at 9:12 AM Ales Musil <[email protected]> wrote: > > > On Thu, Nov 20, 2025 at 9:04 AM Xavier Simonart via dev < > [email protected]> wrote: > >> Health check packets were sent as soon as port_binding chassis was >> set. >> This means that all related flows were potentially not yet installed, >> and health check packet got lost. >> >> Reported-at: https://issues.redhat.com/browse/FDP-2116 > > > nit: This seems to be the wrong number, it should be 2649. > > >> >> Signed-off-by: Xavier Simonart <[email protected]> >> --- >> controller/pinctrl.c | 2 +- >> tests/system-ovn.at | 3 +++ >> 2 files changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/controller/pinctrl.c b/controller/pinctrl.c >> index 7fc111b24..0a8451fc3 100644 >> --- a/controller/pinctrl.c >> +++ b/controller/pinctrl.c >> @@ -7010,7 +7010,7 @@ sync_svc_monitors(struct ovsdb_idl_txn >> *ovnsb_idl_txn, >> continue; >> } >> >> - if (pb->chassis != our_chassis) { >> + if ((pb->chassis != our_chassis) || (pb->n_up && !pb->up[0])) { >> continue; >> } >> >> diff --git a/tests/system-ovn.at b/tests/system-ovn.at >> index d23e4a6aa..332455d0c 100644 >> --- a/tests/system-ovn.at >> +++ b/tests/system-ovn.at >> @@ -4575,9 +4575,12 @@ ovs-vsctl remove interface ovs-sw1-p1 external_ids >> iface-id >> wait_row_count Service_Monitor 2 >> wait_row_count Service_Monitor 2 status=offline >> >> +# Once port is added, status should be online very soon: if it takes >> more than a few seconds, that's a bug. >> +OVS_CTL_TIMEOUT=3 >> ovs-vsctl set interface ovs-sw1-p1 external_ids:iface-id=sw1-p1 >> wait_row_count Service_Monitor 2 >> wait_row_count Service_Monitor 1 status=online >> +OVS_CTL_TIMEOUT=30 >> >> # Create udp load balancer. >> check ovn-nbctl lb-add lb2 10.0.0.10:80 10.0.0.3:80,20.0.0.3:80 udp >> -- >> 2.47.1 >> >> _______________________________________________ >> dev mailing list >> [email protected] >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> >> > Regards, > Ales > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
