On 2/2/22 13:31, Dumitru Ceara wrote:
> It's useful to differentiate between ingress and egress pipelines in the
> ACL logs.  To achieve this we expand the "log()" logical action and pass
> an optional direction.
> 
> This behavior change is implemented in a backwards compatible way such
> that it doesn't break existing deployments even in the case when
> ovn-northd gets updated before ovn-controller.
> 
> To achieve this, ovn-northd determines first if all chassis in the
> cluster have been upgraded to a version that supports the new "log()"
> action format.
> 
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1992641
> Signed-off-by: Dumitru Ceara <[email protected]>
> ---

Looks like I might have overlooked something, this seems to break
ovn-kubernetes' ACL logging use case:

https://github.com/ovsrobot/ovn/runs/5036766769?check_suite_focus=true#step:10:6092

I'll investigate and prepare a v2 once figured out.

Sorry for the noise.


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

Reply via email to