Re: [ovs-dev] [PATCH v3 1/4] Eliminate use of term "slave" in bond, LACP, and bundle contexts.

2020-10-16 Thread Ben Pfaff
On Fri, Jul 10, 2020 at 12:52:12PM +, Alin Serdean wrote: > From: Ben Pfaff<mailto:b...@ovn.org> > Sent: Tuesday, July 7, 2020 7:29 PM > To: d...@openvswitch.org<mailto:d...@openvswitch.org> > Cc: Ben Pfaff<mailto:b...@ovn.org> > Subject: [ovs-dev] [PATCH v3

Re: [ovs-dev] [PATCH v3 1/4] Eliminate use of term "slave" in bond, LACP, and bundle contexts.

2020-10-16 Thread Ben Pfaff
On Mon, Jul 13, 2020 at 07:35:30PM +, Jeff Squyres (jsquyres) wrote: > On Jul 10, 2020, at 8:52 AM, Alin Serdean > wrote: > > > > Most of these changes should not change user-visible behavior. One > > place where they do is in "ovs-ofctl dump-flows", which will now output > > "subs:..." ins

Re: [ovs-dev] [PATCH v3 1/4] Eliminate use of term "slave" in bond, LACP, and bundle contexts.

2020-07-13 Thread Jeff Squyres (jsquyres) via dev
On Jul 10, 2020, at 8:52 AM, Alin Serdean wrote: > > Most of these changes should not change user-visible behavior. One > place where they do is in "ovs-ofctl dump-flows", which will now output > "subs:..." inside "bundle" actions instead of "slaves:...". I don't > expect this to cause real pr

Re: [ovs-dev] [PATCH v3 1/4] Eliminate use of term "slave" in bond, LACP, and bundle contexts.

2020-07-10 Thread Alin Serdean
From: Ben Pfaff<mailto:b...@ovn.org> Sent: Tuesday, July 7, 2020 7:29 PM To: d...@openvswitch.org<mailto:d...@openvswitch.org> Cc: Ben Pfaff<mailto:b...@ovn.org> Subject: [ovs-dev] [PATCH v3 1/4] Eliminate use of term "slave" in bond, LACP, and bundle contexts. Most

Re: [ovs-dev] [PATCH v3 1/4] Eliminate use of term "slave" in bond, LACP, and bundle contexts.

2020-07-07 Thread 0-day Robot
Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 91 characters long (recommended limit is 79) #1866 FILE: lib/lacp.c:1160: lacp_get