On 4/1/26 4:56 PM, Ilya Maximets wrote: > On 4/1/26 2:13 PM, Dumitru Ceara wrote: >> LBs on switches are a broken abstraction that never worked correctly >> and leads to cascading workarounds. Add deprecation warnings in >> ovn-nbctl when attaching a load balancer to a logical switch via >> ls-lb-add, and in ovn-northd when processing a logical switch that >> has load balancers configured. >> >> Reported-by: Ilya Maximets <[email protected]> >> Suggested-by: Ales Musil <[email protected]> >> Signed-off-by: Dumitru Ceara <[email protected]> >> --- >> NEWS | 2 ++ >> northd/northd.c | 9 +++++++++ >> utilities/ovn-nbctl.c | 2 ++ >> 3 files changed, 13 insertions(+) > > Thanks, Dumitru! The patch could use some documentation updates > for the database tables and the warnings could be worded a little > less vaguely, e.g. not use constructs like "a bad thing". But, > otherwise it looks great! :)
Thanks for the review, Ilya! Yeah, the non-RFC version of this needs way more work, I agree. I'll mark this patch as "Deferred" in patchwork until we get time to work on something like this (probably April 1st 2027 :D). Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
