On Tue, Jul 16, 2013 at 12:59 PM, Pavithra Ramesh <[email protected]>wrote:

> This change includes the output of ovs-appctl bfd/show to
> ovs-bugtool output.
>
> Signed-off-by: Pavithra Ramesh <[email protected]>
>
This looks good to me.
I see that there is no documentation of the "bfd/show" command in
ovs-vswitchd man page.
Can you send that as a separate patch?


> ---
>  utilities/bugtool/automake.mk                      |    1 +
>  utilities/bugtool/ovs-bugtool-bfd-show             |   19
> +++++++++++++++++++
>  .../bugtool/plugins/network-status/openvswitch.xml |    1 +
>  xenserver/README                                   |    5 +++++
>  4 files changed, 26 insertions(+), 0 deletions(-)
>  create mode 100755 utilities/bugtool/ovs-bugtool-bfd-show
>
> diff --git a/utilities/bugtool/automake.mk b/utilities/bugtool/automake.mk
> index 38768cb..528b720 100644
> --- a/utilities/bugtool/automake.mk
> +++ b/utilities/bugtool/automake.mk
> @@ -14,6 +14,7 @@ bugtool_plugins = \
>         utilities/bugtool/plugins/system-configuration/openvswitch.xml
>
>  bugtool_scripts = \
> +       utilities/bugtool/ovs-bugtool-bfd-show \
>         utilities/bugtool/ovs-bugtool-cfm-show \
>         utilities/bugtool/ovs-bugtool-coverage-show \
>         utilities/bugtool/ovs-bugtool-lacp-show \
> diff --git a/utilities/bugtool/ovs-bugtool-bfd-show
> b/utilities/bugtool/ovs-bugtool-bfd-show
> new file mode 100755
> index 0000000..26fbda2
> --- /dev/null
> +++ b/utilities/bugtool/ovs-bugtool-bfd-show
> @@ -0,0 +1,19 @@
> +#! /bin/sh
> +
> +# This library is free software; you can redistribute it and/or
> +# modify it under the terms of version 2.1 of the GNU Lesser General
> +# Public License as published by the Free Software Foundation.
> +#
> +# This library is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> +# Lesser General Public License for more details.
> +#
> +# You should have received a copy of the GNU Lesser General Public
> +# License along with this library; if not, write to the Free Software
> +# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
> +# USA
> +#
> +# Copyright (C) 2011 Nicira, Inc.
> +
> +ovs-appctl bfd/show
> diff --git a/utilities/bugtool/plugins/network-status/openvswitch.xml
> b/utilities/bugtool/plugins/network-status/openvswitch.xml
> index 5ca47e6..00247f9 100644
> --- a/utilities/bugtool/plugins/network-status/openvswitch.xml
> +++ b/utilities/bugtool/plugins/network-status/openvswitch.xml
> @@ -22,6 +22,7 @@
>    <command label="dump-ovsdb"
> filters="ovs">/usr/share/openvswitch/scripts/ovs-bugtool-ovsdb-dump</command>
>    <command label="ovs-appctl-lacp-show"
> filters="ovs">/usr/share/openvswitch/scripts/ovs-bugtool-lacp-show</command>
>    <command label="ovs-appctl-cfm-show"
> filters="ovs">/usr/share/openvswitch/scripts/ovs-bugtool-cfm-show</command>
> +  <command label="ovs-appctl-bfd-show"
> filters="ovs">/usr/share/openvswitch/scripts/ovs-bugtool-bfd-show</command>
>    <command label="ovs-appctl-coverage-show"
> filters="ovs">/usr/share/openvswitch/scripts/ovs-bugtool-coverage-show</command>
>    <command label="ovs-appctl-bond-show"
> filters="ovs">/usr/share/openvswitch/scripts/ovs-bugtool-bond-show</command>
>    <command label="ovs-appctl-memory-show"
> filters="ovs">/usr/share/openvswitch/scripts/ovs-bugtool-memory-show</command>
> diff --git a/xenserver/README b/xenserver/README
> index dd49dab..9aedaf6 100644
> --- a/xenserver/README
> +++ b/xenserver/README
> @@ -81,6 +81,7 @@ debugging.  The sources for the extensions are in
>          Collect networking information relevant to Open vSwitch.  Runs
>          the following scripts, which are described below:
>
> +            * ovs-bugtool-bfd-show
>              * ovs-bugtool-cfm-show
>              * ovs-bugtool-lacp-show
>              * ovs-bugtool-list-dbs
> @@ -112,6 +113,10 @@ A number of scripts are installed in
> /usr/share/openvswitch/scripts to
>  assist Open vSwitch's xen-bugtool extensions.  The sources for the
>  scripts are located in ../utilities/bugtool:
>
> +    ovs-bugtool-bfd-show
> +
> +        Script to dump detailed BFD information for all enabled
> interfaces.
> +
>      ovs-bugtool-cfm-show
>
>          Script to dump detailed CFM information for all enabled
> interfaces.
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to