Message ID | 20240315173831.747633-1-amusil@redhat.com |
---|---|
Headers | show |
Series | Remove most of the hardcoded table numbers | expand |
Thanks for the rebase Ales. I merged the entire series to main. On 3/15/24 13:38, Ales Musil wrote: > 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/checkpatch.at | 39 ++++ > 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 +++++++++++++++ > 16 files changed, 698 insertions(+), 335 deletions(-) > create mode 100644 utilities/ovn-debug.8.xml > create mode 100644 utilities/ovn-debug.c >