[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