mbox series

[ovs-dev,v3,0/2] Update and rename dpif_nl_exec_monitor.py.

Message ID cover.1724923203.git.echaudro@redhat.com
Headers show
Series Update and rename dpif_nl_exec_monitor.py. | expand

Message

Eelco Chaudron Aug. 29, 2024, 9:23 a.m. UTC
Updated the dpif_nl_exec_monitor.py utility to also optionally monitor
the other DPIF netlink operations, i.e., DPIF_OP_FLOW_PUT,
DPIF_OP_FLOW_DEL and DPIF_OP_FLOW_GET.

Due to the nature of the change, also renamed the script to
dpif_op_nl_monitor.py.

v2: - Renamed the script.
    - Fixed some general comments from Ilya.
    - Updated the documentation and build files due to the
      script name change.
v3: - Fixed header comments/help text.
    - Fixed spelling mistake
    - Also made the execute operational optional.

Eelco Chaudron (2):
  utilities: Update dpif_nl_exec_monitor.py to include new actions.
  utilities: Updated dpif_nl_exec_monitor.py to debug all operations.

 Documentation/topics/usdt-probes.rst          |   7 +-
 utilities/automake.mk                         |   6 +-
 ..._exec_monitor.py => dpif_op_nl_monitor.py} | 310 +++++++++++++-----
 3 files changed, 233 insertions(+), 90 deletions(-)
 rename utilities/usdt-scripts/{dpif_nl_exec_monitor.py => dpif_op_nl_monitor.py} (69%)

Comments

Aaron Conole Aug. 30, 2024, 1:01 p.m. UTC | #1
Eelco Chaudron <echaudro@redhat.com> writes:

> Updated the dpif_nl_exec_monitor.py utility to also optionally monitor
> the other DPIF netlink operations, i.e., DPIF_OP_FLOW_PUT,
> DPIF_OP_FLOW_DEL and DPIF_OP_FLOW_GET.
>
> Due to the nature of the change, also renamed the script to
> dpif_op_nl_monitor.py.
>
> v2: - Renamed the script.
>     - Fixed some general comments from Ilya.
>     - Updated the documentation and build files due to the
>       script name change.
> v3: - Fixed header comments/help text.
>     - Fixed spelling mistake
>     - Also made the execute operational optional.
>
> Eelco Chaudron (2):
>   utilities: Update dpif_nl_exec_monitor.py to include new actions.
>   utilities: Updated dpif_nl_exec_monitor.py to debug all operations.
>

Thanks for this series, Eelco!

Applied.