Re: [ovs-dev] [PATCH 2/2] vlog: Add vlog/close command.

2016-02-03 Thread Ben Pfaff
On Wed, Jan 27, 2016 at 04:57:41PM -0500, Russell Bryant wrote: > On 12/08/2015 01:59 PM, Ben Pfaff wrote: > > Requested-by: P R Dinesh > > Requested-at: https://github.com/openvswitch/ovs/pull/94 > > Signed-off-by: Ben Pfaff > > Acked-by: Russell Bryant Thanks! I'm going to send a revision of

Re: [ovs-dev] [PATCH 2/2] vlog: Add vlog/close command.

2016-02-03 Thread Ben Pfaff
On Wed, Jan 27, 2016 at 10:54:36PM -0500, Russell Bryant wrote: > On Wednesday, January 27, 2016, Russell Bryant wrote: > > > On 12/08/2015 01:59 PM, Ben Pfaff wrote: > > > Requested-by: P R Dinesh > > > Requested-at: https://github.com/openvswitch/ovs/pull/94 > > > Signed-off-by: Ben Pfaff > > >

Re: [ovs-dev] [PATCH 2/2] vlog: Add vlog/close command.

2016-01-27 Thread Russell Bryant
On Wednesday, January 27, 2016, Russell Bryant wrote: > On 12/08/2015 01:59 PM, Ben Pfaff wrote: > > Requested-by: P R Dinesh > > Requested-at: https://github.com/openvswitch/ovs/pull/94 > > Signed-off-by: Ben Pfaff > > > Acked-by: Russell Bryant > I meant to also ask if you wanted to include t

Re: [ovs-dev] [PATCH 2/2] vlog: Add vlog/close command.

2016-01-27 Thread Russell Bryant
On 12/08/2015 01:59 PM, Ben Pfaff wrote: > Requested-by: P R Dinesh > Requested-at: https://github.com/openvswitch/ovs/pull/94 > Signed-off-by: Ben Pfaff Acked-by: Russell Bryant Some comments/questions inline ... > --- > lib/vlog.c | 24 ++ > python/ovs/vlog.py | 11 +

[ovs-dev] [PATCH 2/2] vlog: Add vlog/close command.

2015-12-08 Thread Ben Pfaff
Requested-by: P R Dinesh Requested-at: https://github.com/openvswitch/ovs/pull/94 Signed-off-by: Ben Pfaff --- lib/vlog.c | 24 ++ python/ovs/vlog.py | 11 +++- tests/vlog.at | 74 ++ 3 files changed, 108 inserti