[ovs-dev] [PATCH ovn v2] ovn-nbctl: Create daemon control socket in ovn run dir

[Ilya Maximets]

On 4/7/20 1:14 PM, numans at ovn.org wrote:
> From: Numan Siddique <numans at ovn.org>
> 
> ovn-nbctl when run as a daemon is creating the ctl socket in
> the ovs rundir. This patch fixes this issue by creating it in
> the ovn rundir.
> 
> When an ovn service is run with -u option (which specifies the
> ctl socket path) and if this path is not absolute, the ovn
> ctl socket path is created in the ovs run dir. This patch
> also fixes this issue by creating it in the ovn run dir.
> 
> Reported-by: Dan Williams <dcbw at redhat.com>
> Signed-off-by: Numan Siddique <numans at ovn.org>
> ---
> v1 -> v2 
> ------
>   * Addressed the review comments from Ilya. If the ctl socket
>     path is specified and if it is not absolute path it is
>     now created in the ovn run dir.
> 
>  controller/ovn-controller.c |  2 +-
>  ic/ovn-ic.c                 | 10 +++-------
>  lib/ovn-util.c              |  9 +++++----
>  lib/ovn-util.h              |  2 +-
>  northd/ovn-northd.c         | 10 +++-------
>  utilities/ovn-nbctl.c       |  6 +++++-
>  6 files changed, 18 insertions(+), 21 deletions(-)

What about ovn-controller-vtep and ovn-trace?