[ovs-dev] [PATCH v5 2/9] ovs-vsctl: Add conntrack zone commands.

[William Tu]

On Fri, Sep 13, 2019 at 10:48 AM Justin Pettit <jpettit at ovn.org> wrote:
>
>
> > On Sep 13, 2019, at 10:41 AM, William Tu <u9012063 at gmail.com> wrote:
> >
> >> Is there a reason you limited this to 18 arguments and not use INT_MAX?
> >
> > I use 18 because at most we have 11 tcp, 3 udp, 2 icmp, total of 16 and plus
> > (dp_name, zone_id), so total is 18.
> > I think using INT_MAX is fine, because at db schema, the value type is set.
>
> That's a fair argument.  However, I'd suggest that we let ovs-vswitchd do that enforcement.  It's nice to decouple the configuration tool from the binary as much as possible in case people run different versions.
>
> > I will merge your diff and send next version.
>
> Sounds good.  I'm hopeful that we'll be able to get this version merged with minimal changes, though.
>
> Thanks,

Hi Justin,

I applied your diff and it's working fine.
I don't have any other change, do you want me to send another
patch or you want to directly apply?

Thanks
William