openstack/devstack
Code Issues Proposed changes

Fix installation of OVS/OVN from sources

Browse Source 
This patch changes user who runs ovsdb-server and ovn-nortd services
to root.
It also adds installation of the libssl dev package before compilation
of the openvswitch if TLS service is enabled.

Co-Authored-By: Fernando Royo <froyo@redhat.com>

Closes-Bug: #1987832
Change-Id: I83fc9250ae5b7c1686938a0dd25d66b40fc6c6aa
This commit is contained in:
Slawek Kaplonski
2022-08-26 12:58:29 +02:00
parent 995c906950
commit 3de92db663
2 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/neutron_plugins/ovn_agent
View File

@@ -616,7 +616,7 @@ function _start_ovs {
dbcmd+=" --remote=db:hardware_vtep,Global,managers $OVS_DATADIR/vtep.db" dbcmd+=" --remote=db:hardware_vtep,Global,managers $OVS_DATADIR/vtep.db"
fi fi
dbcmd+=" $OVS_DATADIR/conf.db" dbcmd+=" $OVS_DATADIR/conf.db"
_run_process ovsdb-server "$dbcmd" _run_process ovsdb-server "$dbcmd" "" "$STACK_GROUP" "root"
# Note: ovn-controller will create and configure br-int once it is started. # Note: ovn-controller will create and configure br-int once it is started.
# So, no need to create it now because nothing depends on that bridge here. # So, no need to create it now because nothing depends on that bridge here.
@@ -704,7 +704,7 @@ function start_ovn {
local cmd="/bin/bash $SCRIPTDIR/ovn-ctl --no-monitor start_northd" local cmd="/bin/bash $SCRIPTDIR/ovn-ctl --no-monitor start_northd"
local stop_cmd="/bin/bash $SCRIPTDIR/ovn-ctl stop_northd" local stop_cmd="/bin/bash $SCRIPTDIR/ovn-ctl stop_northd"
_run_process ovn-northd "$cmd" "$stop_cmd" _run_process ovn-northd "$cmd" "$stop_cmd" "$STACK_GROUP" "root"
else else
_start_process "$OVN_NORTHD_SERVICE" _start_process "$OVN_NORTHD_SERVICE"
fi fi

6
lib/neutron_plugins/ovs_source
View File

@@ -87,9 +87,15 @@ function prepare_for_ovs_compilation {
install_package kernel-devel-$KERNEL_VERSION install_package kernel-devel-$KERNEL_VERSION
install_package kernel-headers-$KERNEL_VERSION install_package kernel-headers-$KERNEL_VERSION
if is_service_enabled tls-proxy; then
install_package openssl-devel
fi
elif is_ubuntu ; then elif is_ubuntu ; then
install_package linux-headers-$KERNEL_VERSION install_package linux-headers-$KERNEL_VERSION
if is_service_enabled tls-proxy; then
install_package libssl-dev
fi
fi fi
} }