Re: [ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-15 Thread Joe Stringer
On 14 October 2015 at 21:55, Justin Pettit wrote: > >> On Oct 14, 2015, at 8:28 PM, Joe Stringer wrote: >> >>> Do you want to make that change too? >> >> Sure. Is it something we want to backport or just apply to master? > > I'd just apply it to master. I don't think we've had any issues with it

Re: [ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-14 Thread Justin Pettit
> On Oct 14, 2015, at 8:28 PM, Joe Stringer wrote: > >> Do you want to make that change too? > > Sure. Is it something we want to backport or just apply to master? I'd just apply it to master. I don't think we've had any issues with it. --Justin

Re: [ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-14 Thread Joe Stringer
On 14 October 2015 at 19:17, Ben Pfaff wrote: > On Wed, Oct 14, 2015 at 05:49:44PM -0700, Joe Stringer wrote: >> On 14 October 2015 at 15:02, Justin Pettit wrote: >> > >> >> On Oct 14, 2015, at 1:27 PM, Joe Stringer wrote: >> >> >> >> When this configuration parameter was initially introduced in

Re: [ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-14 Thread Ben Pfaff
On Wed, Oct 14, 2015 at 05:49:44PM -0700, Joe Stringer wrote: > On 14 October 2015 at 15:02, Justin Pettit wrote: > > > >> On Oct 14, 2015, at 1:27 PM, Joe Stringer wrote: > >> > >> When this configuration parameter was initially introduced into the > >> database, the documentation was not update

Re: [ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-14 Thread Joe Stringer
On 14 October 2015 at 15:02, Justin Pettit wrote: > >> On Oct 14, 2015, at 1:27 PM, Joe Stringer wrote: >> >> When this configuration parameter was initially introduced into the >> database, the documentation was not updated to describe where it >> resides. > > I think we didn't document it on pu

Re: [ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-14 Thread Justin Pettit
> On Oct 14, 2015, at 1:27 PM, Joe Stringer wrote: > > When this configuration parameter was initially introduced into the > database, the documentation was not updated to describe where it > resides. I think we didn't document it on purpose. If we're going to document it, I'd recommend that

Re: [ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-14 Thread Ben Pfaff
On Wed, Oct 14, 2015 at 01:27:00PM -0700, Joe Stringer wrote: > When this configuration parameter was initially introduced into the > database, the documentation was not updated to describe where it > resides. > > Fixes: 72310b041cfa ("upcall: Configure datapath max-idle through ovs-vsctl.") > Rep

[ovs-dev] [PATCH] ovs-vswitchd.conf.db.5: Update docs for max-idle.

2015-10-14 Thread Joe Stringer
When this configuration parameter was initially introduced into the database, the documentation was not updated to describe where it resides. Fixes: 72310b041cfa ("upcall: Configure datapath max-idle through ovs-vsctl.") Reported-by: Hadar Hen Zion Signed-off-by: Joe Stringer --- vswitchd/vswit