openstack/neutron
Code Issues Proposed changes

Always make sure openvswitch kernel module is loaded

Browse Source 
In change https://review.opendev.org/#/c/661065/ we stopped
compiling openvswitch from source, which was always doing
a reload of the kernel module.  We've seen in some cases
the module isn't loaded, so change to always load the
module unconditionally to avoid this.

Change-Id: I2ac2aa4cc84d6f5ac62bc8c7aec67ac17d89c614
Closes-bug: #1845324
This commit is contained in:
Brian Haley 2019-09-25 08:54:45 -04:00
parent 9fdac1e887
commit 6b241bb13c
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
devstack/lib/ovs
View File

@ -27,7 +27,7 @@ function load_module {
if [ "$(trueorfalse True fatal)" == "True" ]; then
sudo modprobe $module || (dmesg && die $LINENO "FAILED TO LOAD $module")
else
sudo modprobe $module || (echo "FAILED TO LOAD vport_geneve" && dmesg)
sudo modprobe $module || (echo "FAILED TO LOAD $module" && dmesg)
fi
}
@ -85,6 +85,12 @@ function prepare_for_compilation {
fi
}
# load_kernel_modules() - load openvswitch kernel module
function load_kernel_modules {
load_module openvswitch
load_module vport-geneve False
dmesg | tail
}
# reload_kernel_modules() - reload openvswitch kernel module
function reload_kernel_modules {
@ -94,9 +100,7 @@ function reload_kernel_modules {
fi
sudo modprobe -r vport_geneve
sudo modprobe -r openvswitch
load_module openvswitch
load_module vport-geneve False
dmesg | tail
load_kernel_modules
}
# compile_ovs() - Compile OVS from source and load needed modules.
@ -136,6 +140,8 @@ function compile_ovs {
if [[ "$build_modules" == "True" ]]; then
sudo make INSTALL_MOD_DIR=kernel/net/openvswitch modules_install
reload_kernel_modules
else
load_kernel_modules
fi
cd $_pwd