Re: [ovs-dev] [PATCH v2 12/26] vconn: Better bundle error management.

2016-07-29 Thread Jarno Rajahalme
> On Jul 29, 2016, at 1:32 PM, Ben Pfaff wrote: > > On Thu, Jul 28, 2016 at 05:56:04PM -0700, Jarno Rajahalme wrote: >> It is possible that a bundle add message fails, but the following >> commit succeeds, since the message was not added to the bundle. Make >> ovs-ofctl fail also

Re: [ovs-dev] [PATCH v2 12/26] vconn: Better bundle error management.

2016-07-29 Thread Ben Pfaff
On Thu, Jul 28, 2016 at 05:56:04PM -0700, Jarno Rajahalme wrote: > It is possible that a bundle add message fails, but the following > commit succeeds, since the message was not added to the bundle. Make > ovs-ofctl fail also in these cases. > > Also, the commit should not be sent if any of the

[ovs-dev] [PATCH v2 12/26] vconn: Better bundle error management.

2016-07-28 Thread Jarno Rajahalme
It is possible that a bundle add message fails, but the following commit succeeds, since the message was not added to the bundle. Make ovs-ofctl fail also in these cases. Also, the commit should not be sent if any of the bundled messages failed. To make sure all the errors are received before