On 8/25/25 5:52 AM, Sragdhara Datta Chaudhuri wrote: > The LB health monitoring functionality has been extended to support NFs. > Network_Function_Group has a list of Network_Functions, each of which has a > reference to network_Function_Health_Check that has the monitoring config. > There is a corresponding SB service_monitor maintaining the online/offline > status. When status changes, northd picks one of the “online” NFs and sets in > network_function_active field of NFG. The redirection rule in LS uses the > ports from this NF. > > Ovn-controller performs the health monitoring by sending ICMP echo request > with source IP and MAC from NB global options “svc_monitor_ip” and > “svc_monitor_mac”, and destination IP and MAC from NB global options > “svc_monitor_ip_dst” and “svc_monitor_mac_dst”. Apart from svc_monitor_mac, > other 3 options are new. The sequence number and id are randomly generated > and stored in service_mon. The NF forwards the same packet out of the other > port. When it comes out, ovn-controller matches the sequence number and id > with stored values and marks online if matched. > > In SB Service_Monitor table three new fields have been added: > type: to indicate “load-balancer” or “network-function” > mac: the destination MAC address for the monitor packets > logical_input_port: The LSP to which the probe packet would be sent > (taken from inport of Network_Function) > In IC SB Service_Monitor table a new field has been added: > type: only allowed value is “load-balancer” > > Co-authored-by: Naveen Yerramneni <naveen.yerramn...@nutanix.com> > Co-authored-by: Karthik Chandrashekar <karthi...@nutanix.com> > Signed-off-by: Naveen Yerramneni <naveen.yerramn...@nutanix.com> > Signed-off-by: Karthik Chandrashekar <karthi...@nutanix.com> > Signed-off-by: Sragdhara Datta Chaudhuri <sragdha.chau...@nutanix.com> > ---
Hi Sragdhara, I didn't review this last patch of the series yet (it's on my list for tomorrow) but I did want to bring to your attention that it causes some of the CI tests to fail ocassionaly (so you might have to run them in a loop locally): 734: ovn-northd.at:10000 Address set incremental processing -- parallelization=yes in: https://github.com/ovsrobot/ovn/actions/runs/17198800734/job/48788917269#step:12:5509 805: ovn-northd.at:17366 sb_cfg propagation -- parallelization=yes in: https://github.com/ovsrobot/ovn/actions/runs/17198800734/job/48788917267#step:12:5464 It would be nice if you could spend some time investigating why this is the case. The failures seem to appear only with this last patch applied. Hopefully we can have the issue that's causing them fixed in v9. In the meantime I'll continue with the review of this last patch of the v8 series. Regards, Dumitru _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
