[ovs-git] [openvswitch/ovs] 7915b9: ovn: Fix some split db issues in ovn-ctl.

GitHub noreply at github.com
Thu Mar 24 23:47:29 UTC 2016


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 7915b9cc3ba59c3584260fa0ab5f16c934d9b873
      https://github.com/openvswitch/ovs/commit/7915b9cc3ba59c3584260fa0ab5f16c934d9b873
  Author: Russell Bryant <russell at ovn.org>
  Date:   2016-03-24 (Thu, 24 Mar 2016)

  Changed paths:
    M ovn/utilities/ovn-ctl

  Log Message:
  -----------
  ovn: Fix some split db issues in ovn-ctl.

The default location for databases should be $dbdir, not $rundir.

The default location for log files comes from $logdir, not $OVS_LOGDIR.

Fix ovs-appctl commands.  The code assumed that passing
--unixctl=ovnnb_db was sufficient to allow an argument of "-t ovnnb_db"
to ovs-appctl.  Unfortunately, that's not the case.  This literally
results in a unix socket called "ovnnb_db" and ovs-appctl will look for
one in the form "onnb_db.PID.ctl".  For now, add ".ctl" to the name and
pass it by full path to ovs-appctl.  In the future, it would be even
better to have an option similar to --unixctl that specifies a prefix
for ".PID.ctl", as I believe that would give the behavior we want.

Reported-by: Gurucharan Shetty <guru at ovn.org>
Reported-at: http://openvswitch.org/pipermail/dev/2016-March/068498.html
Fixes: 60bdd01148e4 ("Separating OVN NB and SB database processes")
Signed-off-by: Russell Bryant <russell at ovn.org>
Acked-by: Gurucharan Shetty <guru at ovn.org>




More information about the git mailing list