mbox series

[ovs-dev,RFC,0/2] OVN: add rate limiting support for controller_events

Message ID cover.1564493246.git.lorenzo.bianconi@redhat.com
Headers show
Series OVN: add rate limiting support for controller_events | expand

Message

Lorenzo Bianconi July 30, 2019, 1:37 p.m. UTC 
Introduce the capability to associate a meter to each controller event
type in order to not overload the pinctrl thread under heavy load.
For each event type we need to define a naming convention for the related
meter:
- empty_lb_backends: event-elb

Lorenzo Bianconi (2):
  OVN: add meter support to trigger_event action
  OVN: northd: add rate limiting support for SB controller events

 include/ovn/actions.h   |  1 +
 ovn/lib/actions.c       | 42 +++++++++++++++++++++++++++++++++------
 ovn/northd/ovn-northd.c | 44 ++++++++++++++++++++++++++++++++---------
 ovn/ovn-nb.xml          |  8 ++++++++
 tests/ovn.at            |  1 +
 5 files changed, 81 insertions(+), 15 deletions(-)

Comments

Numan Siddique July 30, 2019, 2:51 p.m. UTC | #1 
On Tue, Jul 30, 2019 at 7:14 PM Lorenzo Bianconi <
lorenzo.bianconi@redhat.com> wrote:

> Introduce the capability to associate a meter to each controller event
> type in order to not overload the pinctrl thread under heavy load.
> For each event type we need to define a naming convention for the related
> meter:
> - empty_lb_backends: event-elb
>
> Lorenzo Bianconi (2):
>   OVN: add meter support to trigger_event action
>   OVN: northd: add rate limiting support for SB controller events
>

Hi Lorenzo, can you please submit the patches targeting the new repo ?

Thanks
Numan


>
>  include/ovn/actions.h   |  1 +
>  ovn/lib/actions.c       | 42 +++++++++++++++++++++++++++++++++------
>  ovn/northd/ovn-northd.c | 44 ++++++++++++++++++++++++++++++++---------
>  ovn/ovn-nb.xml          |  8 ++++++++
>  tests/ovn.at            |  1 +
>  5 files changed, 81 insertions(+), 15 deletions(-)
>
> --
> 2.21.0
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Lorenzo Bianconi July 30, 2019, 3:08 p.m. UTC | #2 
> On Tue, Jul 30, 2019 at 7:14 PM Lorenzo Bianconi <
> lorenzo.bianconi@redhat.com> wrote:
> 
> > Introduce the capability to associate a meter to each controller event
> > type in order to not overload the pinctrl thread under heavy load.
> > For each event type we need to define a naming convention for the related
> > meter:
> > - empty_lb_backends: event-elb
> >
> > Lorenzo Bianconi (2):
> >   OVN: add meter support to trigger_event action
> >   OVN: northd: add rate limiting support for SB controller events
> >
> 
> Hi Lorenzo, can you please submit the patches targeting the new repo ?

Hi Numan,

ack, will do sending a formal series.

Regards,
Lorenzo

> 
> Thanks
> Numan
> 
> 
> >
> >  include/ovn/actions.h   |  1 +
> >  ovn/lib/actions.c       | 42 +++++++++++++++++++++++++++++++++------
> >  ovn/northd/ovn-northd.c | 44 ++++++++++++++++++++++++++++++++---------
> >  ovn/ovn-nb.xml          |  8 ++++++++
> >  tests/ovn.at            |  1 +
> >  5 files changed, 81 insertions(+), 15 deletions(-)
> >
> > --
> > 2.21.0
> >
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >