Skip to content

Commit

Permalink
xenserver: Update Open vSwitch post upgrade instructions.
Browse files Browse the repository at this point in the history
Currently, when we upgrade the userspace rpm for XenServer,
we print a message asking users to reboot the hypervisor.
This is not needed. The reboot of hypervisor is needed when
we upgrade the rpm containing the kernel module.  This
reboot can sometimes be avoided by running a
"service openvswitch force-reload-kmod".

Signed-off-by: Gurucharan Shetty <gshetty@nicira.com>
  • Loading branch information
shettyg committed Apr 27, 2012
1 parent b5a36e3 commit 43d3667
Showing 1 changed file with 13 additions and 8 deletions.
21 changes: 13 additions & 8 deletions xenserver/openvswitch-xen.spec.in
Original file line number Diff line number Diff line change
Expand Up @@ -291,10 +291,10 @@ done
# Ensure all required services are set to run
for s in openvswitch openvswitch-xapi-update; do
if chkconfig --list $s >/dev/null 2>&1; then
chkconfig --del $s || printf "Could not remove $s init script."
chkconfig --del $s || printf "Could not remove $s init script.\n"
fi
chkconfig --add $s || printf "Could not add $s init script."
chkconfig $s on || printf "Could not enable $s init script."
chkconfig --add $s || printf "Could not add $s init script.\n"
chkconfig $s on || printf "Could not enable $s init script.\n"
done

if [ "$1" = "1" ]; then # $1 = 1 for install
Expand All @@ -307,12 +307,8 @@ else # $1 = 2 for upgrade
printf "\nThe server is not configured to run Open vSwitch. To run in\n"
printf "vswitch mode, you must run the following command:\n\n"
printf "\txe-switch-network-backend vswitch"
else
printf "\nTo use the new Open vSwitch install, you should reboot the\n"
printf "server now. Failure to do so may result in incorrect operation."
printf "\n\n"
fi

printf "\n\n"
fi

%posttrans %{module_package}
Expand All @@ -325,6 +321,15 @@ fi
# be removed.
depmod %{xen_version}

mode=$(cat /etc/xensource/network.conf)
if [ "$mode" = "vswitch" ] || [ "$mode" = "openvswitch" ]; then
printf "\nTo use the newly installed Open vSwitch kernel module, you\n"
printf "will either have to reboot the hypervisor or follow any\n"
printf "workarounds provided by your administration guide. Failure to do\n"
printf "so may result in incorrect operation."
printf "\n\n"
fi

%preun
if [ "$1" = "0" ]; then # $1 = 0 for uninstall
# Configure system to use bridge
Expand Down

0 comments on commit 43d3667

Please sign in to comment.