On 6/5/26 5:42 PM, Mykola Yurchenko via dev wrote:
> In northd, Logical_Switch_Port.tag is set from tag_request.  A
> nonzero request is copied directly into tag.  A request of 0 on a
> child port (one with parent_name set) is replaced by a tag that is
> unique within the parent's scope.
> 
> Currently, tag is not cleared when tag_request is removed, or when
> tag_request=0 on a port without parent_name.  The stale derived tag
> propagates to SB Port_Binding.tag, and ovn-controller keeps tagging
> packets after the CMS removed the request.
> 
> This patch clears the derived tag in both cases.  It also stops
> counting tags as in-use by a child port once their request is gone,
> so the parent's pool is freed correctly.
> 
> Added tests for three transitions: clearing tag_request on a child
> port, clearing tag_request on a non-child port, and setting
> tag_request=0 on a non-child port.
> 
> Signed-off-by: Mykola Yurchenko <[email protected]>
> ---
> v2:
> - Resent with a cleanly generated patch; no code changes from v1.
> 

Hi Mykola,

Thanks for the fix!  Applied to main and 26.03.  I also added you to
OVN's AUTHORS.rst file.

Best regards,
Dumitru

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

Reply via email to