On Wed, Feb 25, 2026 at 8:52 PM Frode Nordahl <[email protected]> wrote:
> On 2/25/26 08:49, Panos Kostopoulos Kyrimis via dev wrote:
> > The purpose of run_* commands is to start the service and never detach.
> > In this case run_ovnbr_ovsdb was detaching due to a typo in the variable
> > being set as no (meaning do not detach).
> >
> > Fixes: 22dbd8021359 ("ovn-ctl: Add commands to start OVN bridge
> controller services.")
> > Reported-at: https://bugs.launchpad.net/bugs/2142198
> > Signed-off-by: Panos Kostopoulos Kyrimis <
> [email protected]>
>
>
> Hello, Panos, and thanks for the patch!
>
> I went ahead and tried this out, and it works as expected.
>
> Reviewed-by: Frode Nordahl <[email protected]>
>
> --
> Frode Nordahl
>
> > ---
> > utilities/ovn-ctl | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/utilities/ovn-ctl b/utilities/ovn-ctl
> > index 2764c714c..3b62ca9b7 100755
> > --- a/utilities/ovn-ctl
> > +++ b/utilities/ovn-ctl
> > @@ -559,7 +559,7 @@ run_ic_sb_ovsdb() {
> > }
> >
> > run_ovnbr_ovsdb() {
> > - DB_NB_DETACH=no
> > + DB_OVNBR_DETACH=no
> > start_ovnbr_ovsdb
> > }
> >
> > --
> > 2.43.0
> >
> > _______________________________________________
> > dev mailing list
> > [email protected]
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
Thank you Panos and Frode,
I went ahead, merged this into main and backported it to 26.03. I've also
added Panos to the AUTHORS file.
Regards,
Ales
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
