On 6/25/21 1:50 PM, Dumitru Ceara wrote:
> It's valid that port_groups contain non-vif ports, they can actually
> contain any type of logical_switch_port.
> 
> Also, there's no need to allocate a new sset containing the local ports'
> names every time the I-P engine processes a change, we can maintain a
> sset and incrementally update it when port bindings are added/removed.
> 
> Reported-at: https://github.com/ovn-org/ovn/pull/61#issuecomment-865094163
> Reported-by: Antonio Ojea <ao...@redhat.com>
> Fixes: 0cfeba6b55e3 ("ovn-controller: Fix port group conjunction flow 
> explosion problem.")
> Signed-off-by: Dumitru Ceara <dce...@redhat.com>
> ---

Hi Han,

It would be great if you could have a look at this patch, commit
0cfeba6b55e3 ("ovn-controller: Fix port group conjunction flow explosion
problem.") breaks ACL use cases in ovn-kubernetes.

Thanks,
Dumitru

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to