diff options
author | Justin Pettit <jpettit@ovn.org> | 2018-07-01 17:58:59 -0700 |
---|---|---|
committer | Justin Pettit <jpettit@ovn.org> | 2018-07-06 14:47:10 -0700 |
commit | 97b089eb540a762edce4adbd5077f9527e26d8ae (patch) | |
tree | 9e4f41051a1799d0b27ee63896942e963dbe34ce | |
parent | c8e0d9809c2681c0f4e34561a434b535f517eefb (diff) | |
download | openvswitch-97b089eb540a762edce4adbd5077f9527e26d8ae.tar.gz |
ovs-sandbox: Fix ovs-appctl for ovn-northd and ovn-controller.
Commits 1e8eeb66db2e7 ("ovs-sandbox: Support starting multiple
ovn-northds.") and 047458de40391 ("ovs-sandbox: Add option to support
multiple ovn-controllers.") allowed starting multiple instances of
ovn-northd and ovn-controller, respectively. It did this by assigning a
sequence number to to the pidfile name. Unfortunately, this breaks the
method ovs-appctl uses to determine to which process it should connect.
This commit changes the behavior so that a sequence number is not added
to the first instance, so ovs-appctl will connect to that be default.
This commit also uses the same convention for naming the log file.
Signed-off-by: Justin Pettit <jpettit@ovn.org>
Acked-by: Ben Pfaff <blp@ovn.org>
-rwxr-xr-x | tutorial/ovs-sandbox | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/tutorial/ovs-sandbox b/tutorial/ovs-sandbox index 3fab1c0cc..62ec537e8 100755 --- a/tutorial/ovs-sandbox +++ b/tutorial/ovs-sandbox @@ -529,16 +529,18 @@ if $ovn; then OVN_CTRLR_PKI="" fi for i in $(seq $n_northds); do + if [ $i -eq 1 ]; then inst=""; else inst=$i; fi rungdb $gdb_ovn_northd $gdb_ovn_northd_ex ovn-northd --detach \ - --no-chdir --pidfile=ovn-northd$i.pid -vconsole:off \ - --log-file=ovn-northd$i.log -vsyslog:off \ + --no-chdir --pidfile=ovn-northd${inst}.pid -vconsole:off \ + --log-file=ovn-northd${inst}.log -vsyslog:off \ --ovnsb-db="$OVN_SB_DB" --ovnnb-db="$OVN_NB_DB" done for i in $(seq $n_controllers); do + if [ $i -eq 1 ]; then inst=""; else inst=$i; fi rungdb $gdb_ovn_controller $gdb_ovn_controller_ex ovn-controller \ $OVN_CTRLR_PKI --detach --no-chdir -vsyslog:off \ - --log-file=ovn-controller$i.log \ - --pidfile=ovn-controller$i.pid -vconsole:off + --log-file=ovn-controller${inst}.log \ + --pidfile=ovn-controller${inst}.pid -vconsole:off done rungdb $gdb_ovn_controller_vtep $gdb_ovn_controller_vtep_ex \ ovn-controller-vtep --detach --no-chdir --pidfile -vconsole:off \ |