mbox series

[ovs-dev,v3,0/4] Remove most of the hardcoded table numbers

Message ID 20240208181719.224501-1-amusil@redhat.com
Headers show
Series Remove most of the hardcoded table numbers | expand

Message

Ales Musil Feb. 8, 2024, 6:17 p.m. UTC 
This series has the remaing patches that weren't yet
accepted with addition that fixes table numbers in comments.

A lot of tests were relying on exact table numbers for
flow comparison either logical or OpenFlows.

In order to take care of table references into OpenFlow through logical
flow stage, new binary is added called ovn-debug. The new binary provides
command that will translate stage name into OpenFlow table number.

Only test that is still hardcoded is "action parsing", because
it compares the direct output from the test binary.

Ales Musil (4):
  tests: Remove hardcoded numbers from comments
  checkpatch: Add rule to check for hardcoded table numbers.
  utilities: Add ovn-debug binary tool.
  tests: Use the ovn-debug binary to determine table numbers.

 NEWS                       |   5 +
 README.rst                 |   1 +
 debian/ovn-common.install  |   1 +
 debian/ovn-common.manpages |   1 +
 rhel/ovn-fedora.spec.in    |   2 +
 tests/ovn-controller.at    | 342 ++++++++++++++++++--------------
 tests/ovn-northd.at        |   2 +-
 tests/ovn.at               | 397 +++++++++++++++++++++----------------
 tests/system-ovn-kmod.at   |  16 +-
 tests/system-ovn.at        |  20 +-
 utilities/.gitignore       |   2 +
 utilities/automake.mk      |  10 +-
 utilities/checkpatch.py    |  12 ++
 utilities/ovn-debug.8.xml  |  28 +++
 utilities/ovn-debug.c      | 155 +++++++++++++++
 15 files changed, 659 insertions(+), 335 deletions(-)
 create mode 100644 utilities/ovn-debug.8.xml
 create mode 100644 utilities/ovn-debug.c