[ovs-dev] [PATCH ovn] ovn-ctl: Create etcdir when starting ovsdb servers

[Numan Siddique]

On Sun, Oct 13, 2019 at 4:31 AM aginwala <aginwala at asu.edu> wrote:

> Acked-by: Aliasgar Ginwala <aginwala at ebay.com <gvrose8192 at gmail.com>>
>

Thanks for the review. I applied this to master.

Numan


>
> On Sat, Oct 12, 2019 at 1:16 PM <nusiddiq at redhat.com> wrote:
>
>> From: Numan Siddique <nusiddiq at 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 <nusiddiq at 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
>> dev at openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
>