Looks good to me.  Thanks.

--Justin


On Mar 22, 2012, at 10:20 AM, Ben Pfaff wrote:

> Commit 24e81092a1 (debian: Bring Debian packaging in-line with new file
> locations) introduced an ambiguous "--t" option invoking ovs-appctl, so
> ovs-vswitchd and ovsdb-server were not reopening their log files following
> log rotation.  This fixes the problem by correct the option name.
> 
> Reported-by: Paul Ingram <p...@nicira.com>
> Signed-off-by: Ben Pfaff <b...@nicira.com>
> ---
> debian/openvswitch-switch.logrotate |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/debian/openvswitch-switch.logrotate 
> b/debian/openvswitch-switch.logrotate
> index d72d230..6620a09 100644
> --- a/debian/openvswitch-switch.logrotate
> +++ b/debian/openvswitch-switch.logrotate
> @@ -8,10 +8,10 @@
>     postrotate
>     # Tell Open vSwitch daemons to reopen their log files
>     if [ -e /var/run/openvswitch/ovs-vswitchd.pid ]; then
> -        ovs-appctl --t ovs-vswitchd vlog/reopen
> +        ovs-appctl -t ovs-vswitchd vlog/reopen
>     fi
>     if [ -e /var/run/openvswitch/ovsdb-server.pid ]; then
> -        ovs-appctl --t ovsdb-server vlog/reopen
> +        ovs-appctl -t ovsdb-server vlog/reopen
>     fi
>     endscript
> }
> -- 
> 1.7.2.5
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to