mbox series

[ovs-dev,0/7] user defined events and notification

Message ID cover.1591091223.git.gmuthukr@redhat.com
Headers show
Series user defined events and notification | expand

Message

Gowrishankar Muthukrishnan June 2, 2020, 10:09 a.m. UTC
This patchset enables new feature "user defined events and notification".
In short, by this feature, one can enable asynchronous and less intrusive
monitoring in Openvswitch data plane and control plane resources, by
defining one or more events to populate inside Openvswitch. Based on the
type of the event, their behaviour is defined (to wait for some condition
be met or to inform in work flow). More information on how to use this
feature is in Documentation/topics/user-defined-events.rst.

This feature is targeting troubleshooting and live debugging situations
in production deployment, hence more helpful for operators and developers
debugging functional and performance issues in Openvswitch. This proposal
is not extensively coveraing all traceable places in Openvswitch, but 
demonstrating how it can be helpful and be extended based on the usecases.
Based on how it is perceived, I can work on enhancing this for optimations
whereever needed.

Gowrishankar Muthukrishnan (7):
  event: add api to manage user defined events
  stopwatch: expose required api for event library
  dpif: hook timer event api in datapath functions
  coverage: support conditional notification from events
  event: test daemon to receive notification from event api
  event: documentation notes on event library
  system-event: add event testsuite

 Documentation/automake.mk                    |   1 +
 Documentation/topics/index.rst               |   1 +
 Documentation/topics/user-defined-events.rst | 306 +++++++++++
 lib/automake.mk                              |   2 +
 lib/coverage.c                               |  61 +++
 lib/coverage.h                               |   4 +
 lib/dpif-netdev.c                            |   5 +-
 lib/event.c                                  | 764 +++++++++++++++++++++++++++
 lib/event.h                                  | 269 ++++++++++
 lib/netdev-dpdk.c                            |   3 +
 lib/stopwatch.c                              |  20 +-
 lib/stopwatch.h                              |   9 +
 lib/timeval.c                                |   3 +
 tests/automake.mk                            |  18 +
 tests/system-event.at                        | 351 ++++++++++++
 utilities/automake.mk                        |   6 +-
 utilities/ovs-testeventd.c                   | 156 ++++++
 vswitchd/bridge.c                            |   2 +
 18 files changed, 1978 insertions(+), 3 deletions(-)
 create mode 100644 Documentation/topics/user-defined-events.rst
 create mode 100644 lib/event.c
 create mode 100644 lib/event.h
 create mode 100644 tests/system-event.at
 create mode 100644 utilities/ovs-testeventd.c

Comments

Gowrishankar Muthukrishnan June 19, 2020, 12:15 p.m. UTC | #1
Hi,
Just curious to probe you again if you had a chance to check this feature.

Thanks in advance.

On Tue, Jun 2, 2020 at 3:40 PM Gowrishankar Muthukrishnan <
gmuthukr@redhat.com> wrote:

> This patchset enables new feature "user defined events and notification".
> In short, by this feature, one can enable asynchronous and less intrusive
> monitoring in Openvswitch data plane and control plane resources, by
> defining one or more events to populate inside Openvswitch. Based on the
> type of the event, their behaviour is defined (to wait for some condition
> be met or to inform in work flow). More information on how to use this
> feature is in Documentation/topics/user-defined-events.rst.
>
> This feature is targeting troubleshooting and live debugging situations
> in production deployment, hence more helpful for operators and developers
> debugging functional and performance issues in Openvswitch. This proposal
> is not extensively coveraing all traceable places in Openvswitch, but
> demonstrating how it can be helpful and be extended based on the usecases.
> Based on how it is perceived, I can work on enhancing this for optimations
> whereever needed.
>
> Gowrishankar Muthukrishnan (7):
>   event: add api to manage user defined events
>   stopwatch: expose required api for event library
>   dpif: hook timer event api in datapath functions
>   coverage: support conditional notification from events
>   event: test daemon to receive notification from event api
>   event: documentation notes on event library
>   system-event: add event testsuite
>
>  Documentation/automake.mk                    |   1 +
>  Documentation/topics/index.rst               |   1 +
>  Documentation/topics/user-defined-events.rst | 306 +++++++++++
>  lib/automake.mk                              |   2 +
>  lib/coverage.c                               |  61 +++
>  lib/coverage.h                               |   4 +
>  lib/dpif-netdev.c                            |   5 +-
>  lib/event.c                                  | 764
> +++++++++++++++++++++++++++
>  lib/event.h                                  | 269 ++++++++++
>  lib/netdev-dpdk.c                            |   3 +
>  lib/stopwatch.c                              |  20 +-
>  lib/stopwatch.h                              |   9 +
>  lib/timeval.c                                |   3 +
>  tests/automake.mk                            |  18 +
>  tests/system-event.at                        | 351 ++++++++++++
>  utilities/automake.mk                        |   6 +-
>  utilities/ovs-testeventd.c                   | 156 ++++++
>  vswitchd/bridge.c                            |   2 +
>  18 files changed, 1978 insertions(+), 3 deletions(-)
>  create mode 100644 Documentation/topics/user-defined-events.rst
>  create mode 100644 lib/event.c
>  create mode 100644 lib/event.h
>  create mode 100644 tests/system-event.at
>  create mode 100644 utilities/ovs-testeventd.c
>
> --
> 1.8.3.1
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
Gowrishankar Muthukrishnan July 14, 2020, 5:09 a.m. UTC | #2
Any suggestion is highly appreciated.

On Tue, Jun 2, 2020 at 3:40 PM Gowrishankar Muthukrishnan <
gmuthukr@redhat.com> wrote:

> This patchset enables new feature "user defined events and notification".
> In short, by this feature, one can enable asynchronous and less intrusive
> monitoring in Openvswitch data plane and control plane resources, by
> defining one or more events to populate inside Openvswitch. Based on the
> type of the event, their behaviour is defined (to wait for some condition
> be met or to inform in work flow). More information on how to use this
> feature is in Documentation/topics/user-defined-events.rst.
>
> This feature is targeting troubleshooting and live debugging situations
> in production deployment, hence more helpful for operators and developers
> debugging functional and performance issues in Openvswitch. This proposal
> is not extensively coveraing all traceable places in Openvswitch, but
> demonstrating how it can be helpful and be extended based on the usecases.
> Based on how it is perceived, I can work on enhancing this for optimations
> whereever needed.
>
> Gowrishankar Muthukrishnan (7):
>   event: add api to manage user defined events
>   stopwatch: expose required api for event library
>   dpif: hook timer event api in datapath functions
>   coverage: support conditional notification from events
>   event: test daemon to receive notification from event api
>   event: documentation notes on event library
>   system-event: add event testsuite
>
>  Documentation/automake.mk                    |   1 +
>  Documentation/topics/index.rst               |   1 +
>  Documentation/topics/user-defined-events.rst | 306 +++++++++++
>  lib/automake.mk                              |   2 +
>  lib/coverage.c                               |  61 +++
>  lib/coverage.h                               |   4 +
>  lib/dpif-netdev.c                            |   5 +-
>  lib/event.c                                  | 764
> +++++++++++++++++++++++++++
>  lib/event.h                                  | 269 ++++++++++
>  lib/netdev-dpdk.c                            |   3 +
>  lib/stopwatch.c                              |  20 +-
>  lib/stopwatch.h                              |   9 +
>  lib/timeval.c                                |   3 +
>  tests/automake.mk                            |  18 +
>  tests/system-event.at                        | 351 ++++++++++++
>  utilities/automake.mk                        |   6 +-
>  utilities/ovs-testeventd.c                   | 156 ++++++
>  vswitchd/bridge.c                            |   2 +
>  18 files changed, 1978 insertions(+), 3 deletions(-)
>  create mode 100644 Documentation/topics/user-defined-events.rst
>  create mode 100644 lib/event.c
>  create mode 100644 lib/event.h
>  create mode 100644 tests/system-event.at
>  create mode 100644 utilities/ovs-testeventd.c
>
> --
> 1.8.3.1
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>