[ovs-dev] [PATCH] xenserver: Add "reload" and "force-reload" support to init script.
Justin Pettit
jpettit at nicira.com
Mon Aug 2 22:31:49 UTC 2010
This is the kind of work that earns you your reputation as a "super-duper hacker".
--Justin
On Aug 2, 2010, at 3:22 PM, Ben Pfaff wrote:
> Oops, the first version failed to update the "help" message.
>
> --8<--------------------------cut here-------------------------->8--
>
> I can't easily find anything that documents what commands Fedora init
> scripts should support, but many of them support "reload" and
> "force-reload". This commit adds support for them to the XenServer init
> scripts. (The Debian init scripts already had support.)
>
> Debian does document that reload and force-reload should be supported:
> http://www.debian.org/doc/debian-policy/ch-opersys.html#s-writing-init
>
> Reported-by: Reid Price <reid at nicira.com>
> Bug #3266.
> ---
> xenserver/etc_init.d_openvswitch | 6 +++++-
> 1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/xenserver/etc_init.d_openvswitch b/xenserver/etc_init.d_openvswitch
> index 36d0afd..0988fcc 100755
> --- a/xenserver/etc_init.d_openvswitch
> +++ b/xenserver/etc_init.d_openvswitch
> @@ -359,6 +359,10 @@ case "$1" in
> restart)
> restart
> ;;
> + reload|force-reload)
> + # Nothing to do--ovs-vswitchd and ovsdb-server keep their configuration
> + # up-to-date all the time.
> + ;;
> strace-vswitchd)
> shift
> strace -p $(cat "$VSWITCHD_PIDFILE") "$@"
> @@ -379,7 +383,7 @@ case "$1" in
> /usr/sbin/ovs-brcompatd -V
> ;;
> help)
> - printf "openvswitch [start|stop|restart|unload|status|version]\n"
> + printf "openvswitch [start|stop|restart|reload|force-reload|status|version]\n"
> ;;
> *)
> printf "Unknown command: $1\n"
> --
> 1.7.1
>
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
More information about the dev
mailing list