On Sat, Sep 26, 2026 at 2:27 AM Ihar Hrachyshka <[email protected]>
wrote:
> bindir is imported from ovs-lib and may point to a separate Open
> vSwitch installation.
>
> Most installations put OVS and OVN under the same prefix. In those
> environments bindir and ovn_bindir are identical, hiding the problem.
> They can differ when packages use separate prefixes or override
> OVS_BINDIR and OVN_BINDIR independently.
>
> Fixes: 2cfaaa4a70ab ("Add support for using OVN specific rundirs")
> Signed-off-by: Ihar Hrachyshka <[email protected]>
> ---
> utilities/ovn-lib.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/utilities/ovn-lib.in b/utilities/ovn-lib.in
> index 5a0766816..8e8eedeab 100644
> --- a/utilities/ovn-lib.in
> +++ b/utilities/ovn-lib.in
> @@ -192,7 +192,7 @@ stop_ovn_daemon () {
> case $action in
> EXIT)
> action "Exiting $1 ($pid)" \
> - ${bindir}/ovn-appctl -T 1 -t $ctl_file exit
> $other_args
> + ${ovn_bindir}/ovn-appctl -T 1 -t $ctl_file
> exit $other_args
> # The above command could have resulted in delayed
> # daemon segfault. And if a monitor is running, it
> # would restart the daemon giving it a new pid.
> --
> 2.54.0
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
Thank you Ihar,
applied to main and backported down to 26.03.
Regards,
Ales
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
