Hi Sragdhara, For now just a few design questions on this patch, as I'm still reviewing the ones before. Please see below.
On 8/20/25 3:25 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_ip4” and > “svc_monitor_mac”, and destination IP and MAC from new NB global options > “svc_monitor_ip4_dst” and “svc_monitor_mac_dst”. The sequence number and id Why restrict only to IPv4? Why does this have to be a NB Global option? Shouldn't it instead be configured in NB.Network_Function_Health_Check records? What if not all network functions run on the same host and use different IPs? Another possible deployment is with multiple OVN switches using individual Network Functions running on different overlays. The restriction of a single NB_Global IP for NF health check monitoring seems very drastic then. > are randomly generated and stored in service_mon. The NF VM 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) > > 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> > --- Regards, Dumitru _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
