[ovs-dev] [PATCH 0/2] Read only versions of *ctl commands

Ryan Moats rmoats at us.ibm.com
Tue Aug 2 04:20:19 UTC 2016


This patch series adds read-only wrapper scripts around the various
*ctl commands that leverage the --dry-run command line option.
This option is added to commands that don't currently have it
(ovs-dpctl and ovs-ofctl) and negative tests are included to verify
that the wrapper scripts don't change things.

Ryan Moats (2):
  Add dry-run option to ovs-dpctl and ovs-ofctl commands.
  Add wrapper scripts for *ctl commands

 lib/command-line.c             |  50 +++++++++----
 lib/command-line.h             |   6 +-
 lib/db-ctl-base.h              |   2 +-
 lib/dpctl.c                    |  44 ++++++------
 lib/dpctl.h                    |   3 +
 ovn/utilities/automake.mk      |   8 ++-
 ovn/utilities/ovn-nbread       |   2 +
 ovn/utilities/ovn-sbread       |   2 +
 ovsdb/ovsdb-tool.c             |  28 ++++----
 tests/ovn-nbctl.at             | 103 ++++++++++++++++++++++++++
 tests/ovn-sbctl.at             |  46 ++++++++++++
 tests/ovs-ofctl.at             |  33 +++++++++
 tests/ovs-vsctl.at             |  90 +++++++++++++++++++++++
 tests/ovstest.c                |   4 +-
 tests/test-bitmap.c            |   6 +-
 tests/test-ccmap.c             |   6 +-
 tests/test-classifier.c        |  28 ++++----
 tests/test-cmap.c              |   6 +-
 tests/test-conntrack.c         |   6 +-
 tests/test-heap.c              |  14 ++--
 tests/test-jsonrpc.c           |  10 +--
 tests/test-netlink-conntrack.c |   8 +--
 tests/test-ovn.c               |  24 +++----
 tests/test-ovsdb.c             |  78 ++++++++++----------
 tests/test-reconnect.c         |  32 ++++-----
 tests/test-util.c              |  34 ++++-----
 tests/test-vconn.c             |  18 ++---
 tests/vtep-ctl.at              | 145 +++++++++++++++++++++++++++++++++++++
 utilities/automake.mk          |  10 ++-
 utilities/ovs-appsetlog        |  37 ++++++++++
 utilities/ovs-dpctl.c          |   7 ++
 utilities/ovs-dpread           |   2 +
 utilities/ovs-ofctl.8.in       |   3 +
 utilities/ovs-ofctl.c          | 160 ++++++++++++++++++++++-------------------
 utilities/ovs-ofread           |   2 +
 utilities/ovs-vsread           |   2 +
 vtep/automake.mk               |   5 +-
 vtep/vtep-read                 |   2 +
 38 files changed, 801 insertions(+), 265 deletions(-)
 create mode 100755 ovn/utilities/ovn-nbread
 create mode 100755 ovn/utilities/ovn-sbread
 create mode 100755 utilities/ovs-appsetlog
 create mode 100755 utilities/ovs-dpread
 create mode 100755 utilities/ovs-ofread
 create mode 100755 utilities/ovs-vsread
 create mode 100755 vtep/vtep-read

-- 
1.9.1




More information about the dev mailing list