On 2/17/26 4:25 PM, Erlon Rodrigues Cruz wrote:
>>>  /* Use common zone for SNAT and DNAT if this option is set to "true". */
>>> -static bool use_common_zone = false;
>>> +static bool use_common_zone;
>> While I like this change, it's unrelated.  It shouldn't be part of this
>> patch.
>>
>>>  /* If this option is 'true' northd will make use of ct.inv match fields.
>>>   * Otherwise, it will avoid using it.  The default is true. */
>>>  static bool use_ct_inv_match = true;
>>>
>>> +/* If this option is 'true' northd will flag the related ACL flows to
>> use
>>> + * connection tracking fields to properly handle IP fragments. By
>> default this
>>> + * option is set to 'false'.
>>> + */
>>> +static bool acl_ct_translation = false;
>>> +
> This and use_common_zone slipped through the cracks. Please let me know if
> there's something else other than those and I add to a v6.

Hi Erlon,

There's no need for a v6 just for these.  I think we (the maintainers)
can address that when merging v5 if the rest is OK.

Thanks,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to