On 7/14/21 10:33 AM, Lorenzo Bianconi wrote:
> This series respin CoPP support introduced here [0] by Dumitru rebasing on top
> of ovn master branch and adding some missing meters (e.g. bfd or acl reject).
> The main goal of this series is to continue the discussion about the proposed
> approach and to align on CMS APIs.
> For the moment DDLog is not supported yet and it will be added in a subsequent
> series.
>
> Related bz:
> https://bugzilla.redhat.com/show_bug.cgi?id=1947913
> https://bugzilla.redhat.com/show_bug.cgi?id=1946610
>
Hi Lorenzo,
First of all, many thanks for picking up this work and moving it forward!
This needs quite some rebase due to 0b201974c8cf ("ovn-northd: Populate
in_out_port in logical_flow table's tags.") being merged before your series.
As far as I see all patches had already been acked (except for 1/5 which
seems good to go anyway based on Ben's "The series is fine as is."
comment). Just to be sure I acked 1/5 now too.
I went ahead and rebased your v7 on top of current master branch. I
pushed it to my fork here:
https://github.com/dceara/ovn/tree/review-pws253444-copp-v7-rebased
It would be great if you could have another look at it to make sure I
didn't mess up anything with the rebase before posting v8. I did run
OVN tests (with ddlog too) and they passed.
Regards,
Dumitru
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
