[PATCH net-next 0/2] Additional tests for action API

[PATCH net-next 0/2] Additional tests for action API

[Vlad Buslov]

Add two new tests for action create/change code.

Vlad Buslov (2):
  tc-testing: add simple action test to verify batch add cleanup
  tc-testing: add simple action test to verify batch change cleanup

 .../tc-testing/tc-tests/actions/simple.json   | 59 +++++++++++++++++++
 1 file changed, 59 insertions(+)

-- 
2.29.2

[PATCH net-next 1/2] tc-testing: add simple action test to verify batch add cleanup

[Vlad Buslov]

Verify cleanup of failed actions batch add where second action in batch
fails after successful init of first action.

Signed-off-by: Vlad Buslov <vladbu@nvidia.com>
---
 .../tc-testing/tc-tests/actions/simple.json   | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json b/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json
index e15f708b0fa4..d5bcbb919dcc 100644
--- a/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json
+++ b/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json
@@ -175,5 +175,35 @@
         "teardown": [
             "$TC actions flush action simple"
         ]
+    },
+    {
+        "id": "8d07",
+        "name": "Verify cleanup of failed actions batch add",
+        "category": [
+            "actions",
+            "simple"
+        ],
+        "setup": [
+            [
+                "$TC actions flush action simple",
+                0,
+                1,
+                255
+            ],
+            "$TC actions add action simple sdata \"2\" index 2",
+            [
+                "$TC actions add action simple sdata \"1\" index 1 action simple sdata \"2\" index 2",
+                255
+            ],
+            "$TC actions flush action simple"
+        ],
+        "cmdUnderTest": "$TC actions add action simple sdata \"2\" index 2",
+        "expExitCode": "0",
+        "verifyCmd": "$TC actions list action simple",
+        "matchPattern": "action order [0-9]*: Simple <2>.*index 2 ref",
+        "matchCount": "1",
+        "teardown": [
+            "$TC actions flush action simple"
+        ]
     }
 ]
-- 
2.29.2

[PATCH net-next 2/2] tc-testing: add simple action test to verify batch change cleanup

[Vlad Buslov]

Verify cleanup of failed actions batch change where second action in batch
fails after successful init of first action.

Signed-off-by: Vlad Buslov <vladbu@nvidia.com>
---
 .../tc-testing/tc-tests/actions/simple.json   | 29 +++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json b/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json
index d5bcbb919dcc..e0c5f060ccb9 100644
--- a/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json
+++ b/tools/testing/selftests/tc-testing/tc-tests/actions/simple.json
@@ -205,5 +205,34 @@
         "teardown": [
             "$TC actions flush action simple"
         ]
+    },
+    {
+        "id": "a68a",
+        "name": "Verify cleanup of failed actions batch change",
+        "category": [
+            "actions",
+            "simple"
+        ],
+        "setup": [
+            [
+                "$TC actions flush action simple",
+                0,
+                1,
+                255
+            ],
+            [
+                "$TC actions change action simple sdata \"1\" index 1 action simple sdata \"2\" goto chain 42 index 2",
+                255
+            ],
+            "$TC actions flush action simple"
+        ],
+        "cmdUnderTest": "$TC actions add action simple sdata \"1\" index 1",
+        "expExitCode": "0",
+        "verifyCmd": "$TC actions list action simple",
+        "matchPattern": "action order [0-9]*: Simple <1>.*index 1 ref",
+        "matchCount": "1",
+        "teardown": [
+            "$TC actions flush action simple"
+        ]
     }
 ]
-- 
2.29.2

Re: [PATCH net-next 0/2] Additional tests for action API

[Cong Wang]

On Wed, Apr 7, 2021 at 8:46 AM Vlad Buslov <vladbu@nvidia.com> wrote:
>
> Add two new tests for action create/change code.

Acked-by: Cong Wang <cong.wang@bytedance.com>

Thanks.

Re: [PATCH net-next 0/2] Additional tests for action API

[patchwork-bot+netdevbpf]

Hello:

This series was applied to netdev/net-next.git (refs/heads/master):

On Wed, 7 Apr 2021 18:46:41 +0300 you wrote:
> Add two new tests for action create/change code.
> 
> Vlad Buslov (2):
>   tc-testing: add simple action test to verify batch add cleanup
>   tc-testing: add simple action test to verify batch change cleanup
> 
>  .../tc-testing/tc-tests/actions/simple.json   | 59 +++++++++++++++++++
>  1 file changed, 59 insertions(+)

Here is the summary with links:
  - [net-next,1/2] tc-testing: add simple action test to verify batch add cleanup
    https://git.kernel.org/netdev/net-next/c/79749ae19de6
  - [net-next,2/2] tc-testing: add simple action test to verify batch change cleanup
    https://git.kernel.org/netdev/net-next/c/652e3124c3ee

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html