Re: [ovs-dev] [PATCH] lacp: Clarify documentation.

2011-09-01 Thread Ethan Jackson
Thanks for the review, I pushed this. Ethan On Thu, Sep 1, 2011 at 00:11, Ethan Jackson wrote: > On Wed, Aug 31, 2011 at 23:37, Justin Pettit wrote: >> You're taking away the one bit of control that ovs-vswitchd could use at its >> own discretion? > > Haha, I actually remembered why I wrote it

Re: [ovs-dev] [PATCH] lacp: Clarify documentation.

2011-09-01 Thread Ethan Jackson
On Wed, Aug 31, 2011 at 23:37, Justin Pettit wrote: > You're taking away the one bit of control that ovs-vswitchd could use at its > own discretion? Haha, I actually remembered why I wrote it that way. I was planning at some point in time to enable LACP by default in passive mode. If we did th

Re: [ovs-dev] [PATCH] lacp: Clarify documentation.

2011-08-31 Thread Justin Pettit
You're taking away the one bit of control that ovs-vswitchd could use at its own discretion? Looks good. --Justin On Aug 31, 2011, at 3:22 PM, Ethan Jackson wrote: > Requested-by: Dan Wendlandt > --- > vswitchd/vswitch.xml |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > d

[ovs-dev] [PATCH] lacp: Clarify documentation.

2011-08-31 Thread Ethan Jackson
Requested-by: Dan Wendlandt --- vswitchd/vswitch.xml |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml index 6082256..4edc101 100644 --- a/vswitchd/vswitch.xml +++ b/vswitchd/vswitch.xml @@ -635,8 +635,8 @@ connected