Re: [ovs-dev] [PATCH v2 1/9] ovs-vswitchd: Add Datapath, CT_Zone, and CT_Zone_Policy tables.

2019-08-05 Thread Darrell Ball
One comment fix: s/ "min": 0, "max": "65535"}},/ "min": 0, "max": "65536"}},/ On Mon, Aug 5, 2019 at 4:09 PM Darrell Ball wrote: > Thanks for the patch > > I avoided duplicate comments from what Justin suggested > > comments inline > > On Thu, Aug 1, 2019 at 3:08 PM Yi-Hung Wei wrote: > >>

Re: [ovs-dev] [PATCH v2 1/9] ovs-vswitchd: Add Datapath, CT_Zone, and CT_Zone_Policy tables.

2019-08-05 Thread Darrell Ball
Thanks for the patch I avoided duplicate comments from what Justin suggested comments inline On Thu, Aug 1, 2019 at 3:08 PM Yi-Hung Wei wrote: > From: Justin Pettit > > From: Justin Pettit > > Signed-off-by: Justin Pettit > --- > vswitchd/vswitch.ovsschema | 43 +++- >

Re: [ovs-dev] [PATCH v2 1/9] ovs-vswitchd: Add Datapath, CT_Zone, and CT_Zone_Policy tables.

2019-08-05 Thread Yi-Hung Wei
On Fri, Aug 2, 2019 at 11:15 AM Justin Pettit wrote: > > > > On Aug 1, 2019, at 3:07 PM, Yi-Hung Wei wrote: > > > > From: Justin Pettit > > > > From: Justin Pettit > > Can you drop one of these "From:" statements? Otherwise it appears in the > commit message. > > As we discussed off-line,

Re: [ovs-dev] [PATCH v2 1/9] ovs-vswitchd: Add Datapath, CT_Zone, and CT_Zone_Policy tables.

2019-08-02 Thread Justin Pettit
> On Aug 1, 2019, at 3:07 PM, Yi-Hung Wei wrote: > > From: Justin Pettit > > From: Justin Pettit Can you drop one of these "From:" statements? Otherwise it appears in the commit message. As we discussed off-line, can you apply the following diff, which we worked on together along with