diff mbox series

[net-next] selftests/tc-testing: require an up to date iproute2 for blockcast tests

Message ID 20240229143825.1373550-1-pctammela@mojatatu.com
State New
Headers
Series [net-next] selftests/tc-testing: require an up to date iproute2 for blockcast tests |

Commit Message

Pedro Tammela Feb. 29, 2024, 2:38 p.m. UTC 
  Add the dependsOn test check for all the mirred blockcast tests.
It will prevent the issue reported by LKFT which happens when an older
iproute2 is used to run the current tdc.

Tests are skipped if the dependsOn check fails.

Reported-by: Linux Kernel Functional Testing <lkft@linaro.org>
Signed-off-by: Pedro Tammela <pctammela@mojatatu.com>
---
Cc: Naresh Kamboju <naresh.kamboju@linaro.org>
---
 .../selftests/tc-testing/tc-tests/actions/mirred.json      | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Naresh Kamboju March 1, 2024, 12:24 p.m. UTC | #1 
Hi Pedro,

On Thu, 29 Feb 2024 at 20:08, Pedro Tammela <pctammela@mojatatu.com> wrote:
>
> Add the dependsOn test check for all the mirred blockcast tests.
> It will prevent the issue reported by LKFT which happens when an older
> iproute2 is used to run the current tdc.

Thank you for the fix patch.
LKFT tests run on Debian rootfs, Please suggest the packages that are needed
for tc-testing.

>
> Tests are skipped if the dependsOn check fails.

- Naresh
Pedro Tammela March 1, 2024, 12:32 p.m. UTC | #2 
On 01/03/2024 09:24, Naresh Kamboju wrote:
> Hi Pedro,
> 
> On Thu, 29 Feb 2024 at 20:08, Pedro Tammela <pctammela@mojatatu.com> wrote:
>>
>> Add the dependsOn test check for all the mirred blockcast tests.
>> It will prevent the issue reported by LKFT which happens when an older
>> iproute2 is used to run the current tdc.
> 
> Thank you for the fix patch.
> LKFT tests run on Debian rootfs, Please suggest the packages that are needed
> for tc-testing.

I believe testing on net-next should also use (iproute2 + iproute2-next)[0].
For stable the same rule would apply linux-x.y uses the released 
iproute2-x.y.

It's my understanding that some distros ship a slightly older iproute2.
So in order to match the versions you would need to compile it.

> 
>>
>> Tests are skipped if the dependsOn check fails.
> 
> - Naresh


[0]
https://git.kernel.org/pub/scm/network/iproute2/iproute2.git 
https://git.kernel.org/pub/scm/network/iproute2/iproute2-next.git
diff mbox series

Patch

diff --git a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json
index 795cf1ce8af0..b73bd255ea36 100644
--- a/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json
+++ b/tools/testing/selftests/tc-testing/tc-tests/actions/mirred.json
@@ -657,6 +657,7 @@ 
             "actions",
             "mirred"
         ],
+        "dependsOn": "$TC actions add action mirred help 2>&1 | grep -q blockid",
         "plugins": {
             "requires": "nsPlugin"
         },
@@ -711,6 +712,7 @@ 
             "actions",
             "mirred"
         ],
+        "dependsOn": "$TC actions add action mirred help 2>&1 | grep -q blockid",
         "plugins": {
             "requires": "nsPlugin"
         },
@@ -765,6 +767,7 @@ 
             "actions",
             "mirred"
         ],
+        "dependsOn": "$TC actions add action mirred help 2>&1 | grep -q blockid",
         "plugins": {
             "requires": "nsPlugin"
         },
@@ -819,6 +822,7 @@ 
             "actions",
             "mirred"
         ],
+        "dependsOn": "$TC actions add action mirred help 2>&1 | grep -q blockid",
         "plugins": {
             "requires": "nsPlugin"
         },
@@ -873,6 +877,7 @@ 
             "actions",
             "mirred"
         ],
+        "dependsOn": "$TC actions add action mirred help 2>&1 | grep -q blockid",
         "plugins": {
             "requires": "nsPlugin"
         },
@@ -937,6 +942,7 @@ 
             "actions",
             "mirred"
         ],
+        "dependsOn": "$TC actions add action mirred help 2>&1 | grep -q blockid",
         "plugins": {
             "requires": "nsPlugin"
         },
@@ -995,6 +1001,7 @@ 
             "actions",
             "mirred"
         ],
+        "dependsOn": "$TC actions add action mirred help 2>&1 | grep -q blockid",
         "plugins": {
             "requires": "nsPlugin"
         },