Re: [ovs-dev] [PATCH 2/3] ovsdb-idl: Properly handle conditional monitor update error

2017-01-06 Thread Andy Zhou
On Wed, Jan 4, 2017 at 3:55 PM, Ben Pfaff wrote: > On Thu, Dec 22, 2016 at 12:12:05PM -0800, Andy Zhou wrote: > > On Thu, Dec 22, 2016 at 9:21 AM, Ben Pfaff wrote: > > > > > On Tue, Dec 20, 2016 at 01:47:16AM -0800, Andy Zhou wrote: > > > > From: andy zhou

Re: [ovs-dev] [PATCH 2/3] ovsdb-idl: Properly handle conditional monitor update error

2017-01-04 Thread Ben Pfaff
On Thu, Dec 22, 2016 at 12:12:05PM -0800, Andy Zhou wrote: > On Thu, Dec 22, 2016 at 9:21 AM, Ben Pfaff wrote: > > > On Tue, Dec 20, 2016 at 01:47:16AM -0800, Andy Zhou wrote: > > > From: andy zhou > > > > > > When generating conditional monitoring update request,

Re: [ovs-dev] [PATCH 2/3] ovsdb-idl: Properly handle conditional monitor update error

2016-12-22 Thread Ben Pfaff
On Tue, Dec 20, 2016 at 01:47:16AM -0800, Andy Zhou wrote: > From: andy zhou > > When generating conditional monitoring update request, current code > failed to update idl's 'request-id'. This bug causes the reply > message of the update request, regardless an ACK or a NACK, be >

[ovs-dev] [PATCH 2/3] ovsdb-idl: Properly handle conditional monitor update error

2016-12-20 Thread Andy Zhou
From: andy zhou When generating conditional monitoring update request, current code failed to update idl's 'request-id'. This bug causes the reply message of the update request, regardless an ACK or a NACK, be logged as an unexpected message at the debug level and ignored by the