Re: [ovs-dev] [PATCH v2 3/3] ofp-actions: Always consider inconsistent CT actions as an error.

2016-09-27 Thread Jarno Rajahalme
Thanks for the fast review, Joe! Series pushed to master, branch-2.6, and branch-2.5. Jarno > On Sep 27, 2016, at 2:38 PM, Joe Stringer wrote: > > On 27 September 2016 at 14:32, Jarno Rajahalme > wrote: >> We can't downgrade to OF1.0 and

Re: [ovs-dev] [PATCH v2 3/3] ofp-actions: Always consider inconsistent CT actions as an error.

2016-09-27 Thread Joe Stringer
On 27 September 2016 at 14:32, Jarno Rajahalme wrote: > We can't downgrade to OF1.0 and expect inconsistent CT actions > be silently disgarded. Instead, datapath flow install fails, so > it is better to flag inconsistent CT actions as hard errors. > > Signed-off-by: Jarno

[ovs-dev] [PATCH v2 3/3] ofp-actions: Always consider inconsistent CT actions as an error.

2016-09-27 Thread Jarno Rajahalme
We can't downgrade to OF1.0 and expect inconsistent CT actions be silently disgarded. Instead, datapath flow install fails, so it is better to flag inconsistent CT actions as hard errors. Signed-off-by: Jarno Rajahalme --- lib/ofp-actions.c | 7 +--