Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-10 Thread Ben Pfaff
On Wed, May 09, 2012 at 06:34:31AM -0700, Ben Pfaff wrote: > On Tue, May 08, 2012 at 11:47:30PM -0700, Arun Sharma wrote: > > This is an enhancement in bugtool archive output to determine the bond > > state information. It is implemented as a plugin which internally calls > > "ovs-appctl bond/show"

Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-09 Thread Ben Pfaff
On Wed, May 09, 2012 at 11:02:46AM -0700, Ethan Jackson wrote: > > Ethan had complained about the "appctl" missing and asked that we add it > > back (to the new one and change the old ones), which is why Arun did it in > > that way.  I don't have a strong preference.  Ethan, what are your thought

Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-09 Thread Ethan Jackson
> Ethan had complained about the "appctl" missing and asked that we add it back > (to the new one and change the old ones), which is why Arun did it in that > way.  I don't have a strong preference.  Ethan, what are your thoughts? I always get tripped up looking for lacp/show cfm/show etc output

Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-09 Thread Justin Pettit
On May 9, 2012, at 6:34 AM, Ben Pfaff wrote: > On Tue, May 08, 2012 at 11:47:30PM -0700, Arun Sharma wrote: >> This is an enhancement in bugtool archive output to determine the bond >> state information. It is implemented as a plugin which internally calls >> "ovs-appctl bond/show" command to get

Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-09 Thread Ben Pfaff
This seems to me like a waste of time, but at least it shouldn't cause any trouble. On Wed, May 09, 2012 at 10:01:38PM +0530, Arun Sharma wrote: > As per Feature #11283, it was expected to have the label name containing > 'appctl'. > It is also planned to have 'appctl' added for other labels whic

Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-09 Thread Arun Sharma
As per Feature #11283, it was expected to have the label name containing 'appctl'. It is also planned to have 'appctl' added for other labels which uses ovs-appctl commands. On 5/9/12 7:04 PM, "Ben Pfaff" wrote: >On Tue, May 08, 2012 at 11:47:30PM -0700, Arun Sharma wrote: >> This is an enhance

Re: [ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-09 Thread Ben Pfaff
On Tue, May 08, 2012 at 11:47:30PM -0700, Arun Sharma wrote: > This is an enhancement in bugtool archive output to determine the bond > state information. It is implemented as a plugin which internally calls > "ovs-appctl bond/show" command to get bond state. > > Feature #11283 > Signed-off-by: Ar

[ovs-dev] [PATCH] bugtool - Collect bond state information from ovs.

2012-05-09 Thread Arun Sharma
This is an enhancement in bugtool archive output to determine the bond state information. It is implemented as a plugin which internally calls "ovs-appctl bond/show" command to get bond state. Feature #11283 Signed-off-by: Arun Sharma --- utilities/bugtool/automake.mk |3