Re: [ovs-dev] [PATCH v4 ovn 2/4] ovn-northd: Add support for CoPP.

2021-06-18 Thread Lorenzo Bianconi
> On 16/06/2021 11:30, Lorenzo Bianconi wrote: > > +dnl This test is not suported by ovn-northd-ddlog yet. > > +AT_SKIP_IF([test NORTHD_TYPE = ovn-northd-ddlog && test "$RUN_ANYWAY" != > > yes]) > > One more thing .. > > I am unsure what the policy is with regards to pushing this without > ddlog

Re: [ovs-dev] [PATCH v4 ovn 2/4] ovn-northd: Add support for CoPP.

2021-06-18 Thread Mark Gray
On 16/06/2021 11:30, Lorenzo Bianconi wrote: > +dnl This test is not suported by ovn-northd-ddlog yet. > +AT_SKIP_IF([test NORTHD_TYPE = ovn-northd-ddlog && test "$RUN_ANYWAY" != > yes]) One more thing .. I am unsure what the policy is with regards to pushing this without ddlog support. Perhaps

Re: [ovs-dev] [PATCH v4 ovn 2/4] ovn-northd: Add support for CoPP.

2021-06-18 Thread Mark Gray
On 16/06/2021 11:30, Lorenzo Bianconi wrote: > From: Dumitru Ceara > > Add new 'Copp' (Control plane protection) table to OVN Northbound DB: > - this stores mappings between control plane protocol names and meters > that should be used to rate limit controller-destined traffic for > those pro

Re: [ovs-dev] [PATCH v4 ovn 2/4] ovn-northd: Add support for CoPP.

2021-06-16 Thread 0-day Robot
Bleep bloop. Greetings Lorenzo Bianconi, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line lacks whitespace around operator #883 FILE: utilities/ovn-nbctl.c:430: ls-c

[ovs-dev] [PATCH v4 ovn 2/4] ovn-northd: Add support for CoPP.

2021-06-16 Thread Lorenzo Bianconi
From: Dumitru Ceara Add new 'Copp' (Control plane protection) table to OVN Northbound DB: - this stores mappings between control plane protocol names and meters that should be used to rate limit controller-destined traffic for those protocols. Add new 'copp' columns to the following OVN Nort