[ovs-git] [openvswitch/ovs] 2351c7: ovn-ctl: Unify OVN_RUNDIR usage.

Han Zhou noreply at github.com
Fri Mar 22 21:50:01 UTC 2019


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 2351c7fd08946672035df191e3914bbde84e87d1
      https://github.com/openvswitch/ovs/commit/2351c7fd08946672035df191e3914bbde84e87d1
  Author: Han Zhou <hzhou8 at ebay.com>
  Date:   2019-03-22 (Fri, 22 Mar 2019)

  Changed paths:
    M ovn/utilities/ovn-ctl

  Log Message:
  -----------
  ovn-ctl: Unify OVN_RUNDIR usage.

In this script $rundir and $OVN_RUNDIR is used in a mixed way, which
can cause different folders used for different runtime files. This
patch unifies the usage to the correct one.

Signed-off-by: Han Zhou <hzhou8 at ebay.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: 8352f3a847513e5f9a3c9f5b90428eb7e80f2242
      https://github.com/openvswitch/ovs/commit/8352f3a847513e5f9a3c9f5b90428eb7e80f2242
  Author: Han Zhou <hzhou8 at ebay.com>
  Date:   2019-03-22 (Fri, 22 Mar 2019)

  Changed paths:
    M ovn/utilities/ovn-ctl

  Log Message:
  -----------
  ovn-ctl: Make sure OVN_RUNDIR is created for central nodes.

When ovn-ctl tries to start ovsdb, it didn't ensure the rundir
(e.g. /var/run/openvswitch) exist, because it is not calling
start_daemon(). Usually, if OVS is started by ovs-ctl before
on the same node, the folder is created already. However, for
OVN central node, OVS is usually not needed. If the folder is
not created (it is common case when system restarted because
/var/run is usually tmpfs), ovn-ctl will fail to start ovsdb.
This patch always ensures the OVN_RUNDIR is created.

Signed-off-by: Han Zhou <hzhou8 at ebay.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/b16b5caff746...8352f3a84751


More information about the git mailing list