On Wed, Oct 02, 2024 at 09:38:53PM +0200, Phil Sutter wrote: [...] > tests/monitor/testcases/chain-netdev.t | 66 ++++++++++++++++++++++ > tests/monitor/testcases/flowtable-simple.t | 56 ++++++++++++++++++ And as foretold in patch 4, the new chain-netdev.t clashes with similar content in flowtable-simple.t. To avoid this, both need a trivial adjustment: diff --git a/tests/monitor/testcases/chain-netdev.t b/tests/monitor/testcases/chain-netdev.t index 3c004af0cd855..414040045dd20 100644 --- a/tests/monitor/testcases/chain-netdev.t +++ b/tests/monitor/testcases/chain-netdev.t @@ -48,6 +48,7 @@ I add chain netdev t c2 { type filter hook ingress devices = { wald* } priority @ ip link del wild23 I delete chain netdev t c I delete chain netdev t c2 +@ ip link del wald42 O add chain netdev t c { type filter hook ingress devices = { wild* } priority 0; policy accept; } O add chain netdev t c2 { type filter hook ingress devices = { wald* } priority 0; policy accept; } O add device chain netdev t c hook wild* { wild23 } diff --git a/tests/monitor/testcases/flowtable-simple.t b/tests/monitor/testcases/flowtable-simple.t index 113b15f20d1dc..2c82f4907ed93 100644 --- a/tests/monitor/testcases/flowtable-simple.t +++ b/tests/monitor/testcases/flowtable-simple.t @@ -48,6 +48,7 @@ I add flowtable ip t ft2 { hook ingress priority 0; devices = { wald* }; } @ ip link del wild23 I delete flowtable ip t ft I delete flowtable ip t ft2 +@ ip link del wald42 O add flowtable ip t ft { hook ingress priority 0; devices = { wild* }; } O add flowtable ip t ft2 { hook ingress priority 0; devices = { wald* }; } O add device flowtable ip t ft hook wild* { wild23 } I'll wait for feedback before including this in a v2. Sorry for the mess.