On 12/19/25 10:48 AM, Alexandra Rukomoinikova wrote: > 1) Added tables for further implementation logic of service monitors for > logical switch ports: > > New table: > - Logical_Switch_Port_Health_Check: Health check configuration > for logical switch port. > > Modified tables: > - Logical_Switch_Port: Add 'health_checks' column referencing > health checks configuration. > > 2) Added commands to create, delete, and describe health checks for logical > switch ports. > > Signed-off-by: Alexandra Rukomoinikova <[email protected]> > ---
Hi Alexandra, I started reviewing this series but before doing that I want to double check the use case. Is your plan to use these to check the availability of Logical_Switch_Ports as backends of NB.Load_Balancer? If so, how do you plan to add that mapping? I guess it would be a new feature in 26.09. Or is your goal to just expose the LSPHC.status to the CMS and then let the CMS update the list of backends of the LB? In any case we probably should: - update the documentation expanding on the use case more - update the commit log of this patch (and maybe the others too) - add a TODO.rst item for any follow up work we might be doing in 26.09. I have some more comments on the code changes themselves but I'll send those replies separately when I'm done going through the code. Thanks, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
