On Sun, Oct 13, 2019 at 4:31 AM aginwala <aginw...@asu.edu> wrote: > Acked-by: Aliasgar Ginwala <aginw...@ebay.com <gvrose8...@gmail.com>> >
Thanks for the review. I applied this to master. Numan > > On Sat, Oct 12, 2019 at 1:16 PM <nusid...@redhat.com> wrote: > >> From: Numan Siddique <nusid...@redhat.com> >> >> 'ovn-ctl promote_ovnnb/promote_ovnsb' stores the address of the >> active server in /etc/ovn/ovn(nb/sb)-active.conf and this fails as >> ovn-ctl doesn't create /etc/ovn dir. >> >> This patch fixes it. >> >> Signed-off-by: Numan Siddique <nusid...@redhat.com> >> --- >> utilities/ovn-ctl | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/utilities/ovn-ctl b/utilities/ovn-ctl >> index 433ee4f50..481e28fc9 100755 >> --- a/utilities/ovn-ctl >> +++ b/utilities/ovn-ctl >> @@ -153,6 +153,7 @@ start_ovsdb__() { >> ovn_install_dir "$OVN_RUNDIR" >> ovn_install_dir "$ovn_logdir" >> ovn_install_dir "$ovn_dbdir" >> + ovn_install_dir "$ovn_etcdir" >> >> # Check and eventually start ovsdb-server for DB >> if pidfile_is_running $db_pid_file; then >> @@ -194,6 +195,7 @@ $cluster_remote_port >> chown -R $INSTALL_USER:$INSTALL_GROUP $ovn_dbdir >> chown -R $INSTALL_USER:$INSTALL_GROUP $OVN_RUNDIR >> chown -R $INSTALL_USER:$INSTALL_GROUP $ovn_logdir >> + chown -R $INSTALL_USER:$INSTALL_GROUP $ovn_etcdir >> >> set ovsdb-server >> set "$@" $log --log-file=$logfile >> -- >> 2.21.0 >> >> _______________________________________________ >> dev mailing list >> d...@openvswitch.org >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev