Paolo Valerio <[email protected]> writes: > Simon Horman <[email protected]> writes: > >> Hi, >> >> This series aims to update the testsuite so that, if available, >> nft is used in palce of iptables. The motivation being to move >> to more modern tooling. >> >> --- > > Hi Simon,
Hi Paolo, > The patches look good, I also performed some tests and things work as > expected. > > I noticed that "datapath - ping over erspan v1 tunnel by simulated > packets" still uses IPTABLES_ACCEPT(). > > Also, "datapath - ping over erspan v2 tunnel by simulated packets" does > not use the macro, but directly uses iptables with the ACCEPT target > > These last two also: > datapath - ping over ip6erspan v1 tunnel by simulated packets > datapath - ping over ip6erspan v2 tunnel by simulated packets > > instead, do the same but for v6 (ip6tables). > They went unnoticed while adding $HAVE_IPTABLES. > > I guess those should be handled in this set as well. > WDYT? Sorry - I didn't see your comment when doing the apply (my tool didn't pull the cover letter comments). I guess this should be done as a follow up. >> Changes in v2: >> - Drop dependency in v2 >> - I have verified that nft is used when the CI runs the testsuite >> - Link to v1: >> https://mail.openvswitch.org/pipermail/ovs-dev/2024-October/417704.html >> >> --- >> Simon Horman (3): >> tests: add nft accept support. >> tests: Add nft support to ADD_EXTERNAL_CT. >> tests: Handle marks using nft if available. >> >> tests/atlocal.in | 3 ++ >> tests/ovs-macros.at | 26 ++++++++++++- >> tests/system-common-macros.at | 4 ++ >> tests/system-kmod-macros.at | 80 >> +++++++++++++++++++++++++++++++++++++--- >> tests/system-offloads-traffic.at | 29 ++++++++++++++- >> tests/system-traffic.at | 4 +- >> 6 files changed, 135 insertions(+), 11 deletions(-) >> >> base-commit: e998d4558c10938082e02372ac42f828d252c3cd > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
