diff mbox series

[iproute2] testsuite: Fix tc/vlan.t test

Message ID 31ec3c6473b76d320301767cc3920ce6dac33e4c.1641232756.git.aclaudi@redhat.com (mailing list archive)
State Accepted
Commit 1225e307102041fbadfd4e8f073c7e76e74d7561
Delegated to: Stephen Hemminger
Headers show
Series [iproute2] testsuite: Fix tc/vlan.t test | expand

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Andrea Claudi Jan. 3, 2022, 6 p.m. UTC
Following commit 8323b20f1d76 ("net/sched: act_vlan: No dump for unset
priority"), the kernel no longer dump vlan priority if not explicitly
set before.

When modifying a vlan, tc/vlan.t test expects to find priority set to 0
without setting it explicitly. Thus, after 8323b20f1d76 this test fails.

Fix this simply removing the check on priority.

Signed-off-by: Andrea Claudi <aclaudi@redhat.com>
---
 testsuite/tests/tc/vlan.t | 1 -
 1 file changed, 1 deletion(-)

Comments

patchwork-bot+netdevbpf@kernel.org Jan. 8, 2022, 7:40 p.m. UTC | #1
Hello:

This patch was applied to iproute2/iproute2.git (main)
by Stephen Hemminger <stephen@networkplumber.org>:

On Mon,  3 Jan 2022 19:00:22 +0100 you wrote:
> Following commit 8323b20f1d76 ("net/sched: act_vlan: No dump for unset
> priority"), the kernel no longer dump vlan priority if not explicitly
> set before.
> 
> When modifying a vlan, tc/vlan.t test expects to find priority set to 0
> without setting it explicitly. Thus, after 8323b20f1d76 this test fails.
> 
> [...]

Here is the summary with links:
  - [iproute2] testsuite: Fix tc/vlan.t test
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=1225e3071020

You are awesome, thank you!
diff mbox series

Patch

diff --git a/testsuite/tests/tc/vlan.t b/testsuite/tests/tc/vlan.t
index b86dc364..51529b2d 100755
--- a/testsuite/tests/tc/vlan.t
+++ b/testsuite/tests/tc/vlan.t
@@ -50,7 +50,6 @@  test_on "vlan"
 test_on "modify"
 test_on "id 5"
 test_on "protocol 802.1Q"
-test_on "priority 0"
 test_on "pipe"
 
 reset_qdisc