commit b575af0cfe8aac71825b406394e9927a33ca51bd
author yatinkarel <ykarel@redhat.com> Fri Nov 26 12:44:41 2021 +0530
committer yatinkarel <ykarel@redhat.com> Fri Nov 26 13:44:10 2021 +0530
tree db4adf559cdd67d4c8d194b2fc89ecb971738d0d
parent acc9bd6ab921840045f771974d7d4df9bd4dd854

Do not use sudo with OVN_DATADIR when building from source

Jobs with OVN_BUILD_FROMS_SOURCE=True are broken
since [1] as ovn nortd not starting due to permission
issues. Fix it by not using sudo for creating OVN_DATADIR
when building from source.

[1] https://review.opendev.org/c/openstack/devstack/+/806858

Closes-Bug: #1952393
Change-Id: I00f0c8c8173b4d8270fbb3e6079d0d8b332e9de5

lib/neutron_plugins/ovn_agent

diff --git a/lib/neutron_plugins/ovn_agent b/lib/neutron_plugins/ovn_agent
index 999851e..56686f2 100644
--- a/lib/neutron_plugins/ovn_agent
+++ b/lib/neutron_plugins/ovn_agent
@@ -568,7 +568,11 @@
 
     _disable_libvirt_apparmor
 
-    sudo mkdir -p $OVN_DATADIR
+    if [[ "$OVN_BUILD_FROM_SOURCE" == "True" ]]; then
+        mkdir -p $OVN_DATADIR
+    else
+        sudo mkdir -p $OVN_DATADIR
+    fi
     mkdir -p $OVS_DATADIR
 
     rm -f $OVS_DATADIR/*.db