Re: [ovs-dev] [PATCH] ovsdb: Clear 'bfd_status' when bfd is disabled.

2013-09-11 Thread Alex Wang
I see, I'll resend v2 patch. Thanks, On Wed, Sep 11, 2013 at 5:06 PM, Ben Pfaff wrote: > On Wed, Sep 11, 2013 at 04:39:40PM -0700, Alex Wang wrote: > > This commit makes ovsdb clear the 'bfd_status' column > > when bfd is disabled. > > > > Reported-by: Ansis Atteka > > Signed-off-by: Alex Wang

Re: [ovs-dev] [PATCH] ovsdb: Clear 'bfd_status' when bfd is disabled.

2013-09-11 Thread Ben Pfaff
On Wed, Sep 11, 2013 at 04:39:40PM -0700, Alex Wang wrote: > This commit makes ovsdb clear the 'bfd_status' column > when bfd is disabled. > > Reported-by: Ansis Atteka > Signed-off-by: Alex Wang The Subject should refer to "vswitchd:", since this is vswitchd code. ("ovsdb:" would be for chang

Re: [ovs-dev] [PATCH] ovsdb: Clear 'bfd_status' when bfd is disabled.

2013-09-11 Thread Alex Wang
Thanks Ansis, I'll send a separate patch for that. On Wed, Sep 11, 2013 at 5:02 PM, Ansis Atteka wrote: > This looks reasonable to me. One comment above. > > On Wed, Sep 11, 2013 at 4:39 PM, Alex Wang wrote: > > This commit makes ovsdb clear the 'bfd_status' column > > when bfd is disabled. >

Re: [ovs-dev] [PATCH] ovsdb: Clear 'bfd_status' when bfd is disabled.

2013-09-11 Thread Ansis Atteka
This looks reasonable to me. One comment above. On Wed, Sep 11, 2013 at 4:39 PM, Alex Wang wrote: > This commit makes ovsdb clear the 'bfd_status' column > when bfd is disabled. > > Reported-by: Ansis Atteka > Signed-off-by: Alex Wang > --- > vswitchd/bridge.c |5 +++-- > 1 file changed, 3

[ovs-dev] [PATCH] ovsdb: Clear 'bfd_status' when bfd is disabled.

2013-09-11 Thread Alex Wang
This commit makes ovsdb clear the 'bfd_status' column when bfd is disabled. Reported-by: Ansis Atteka Signed-off-by: Alex Wang --- vswitchd/bridge.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index 5e54e0b..f49292b 100644 --