Validate ACL rules and routes in test_everflow_testbed before traffic tests [202511 backport]#23427
Merged
vmittal-msft merged 16 commits intosonic-net:202511from Mar 31, 2026
Conversation
* Strict priority QOS test. Signed-off-by: Abhishek <abhishek@nexthop.ai> * Add q3d and portchannel support for strict priority. Signed-off-by: Abhishek <abhishek@nexthop.ai> * Fix per-commit and missing function. Signed-off-by: Abhishek <abhishek@nexthop.ai> --------- Signed-off-by: Abhishek <abhishek@nexthop.ai>
Signed-off-by: saravanan sellappa <saravanan@nexthop.ai>
…voq duts reliable tsa tests require two duthosts: one as supe and one as LC single asic voq dut have only one duthost, hence reliable tsa tests are skipped backporting sonic-net#22089 Signed-off-by: Saravanan Sellappa <saravanan@nexthop.ai>
Signed-off-by: Venu <venu@nexthop.ai>
This reverts commit 88c7897. Signed-off-by: venu-nexthop <venu@nexthop.ai>
…le asic voq duts" This reverts commit 66748e6. Signed-off-by: venu-nexthop <venu@nexthop.ai>
Revert qos strict prio changes Signed-off-by: Venu <venu@nexthop.ai>
Signed-off-by: venu-nexthop <venu@nexthop.ai>
Signed-off-by: venu-nexthop <venu@nexthop.ai>
Signed-off-by: venu-nexthop <venu@nexthop.ai>
Signed-off-by: venu-nexthop <venu@nexthop.ai>
Signed-off-by: venu-nexthop <venu@nexthop.ai>
Signed-off-by: venu-nexthop <venu@nexthop.ai>
Signed-off-by: venu-nexthop <venu@nexthop.ai>
venu-nexthop
requested review from
BYGX-wcr,
StormLiangMS,
abdosi,
xwjiang-ms and
yutongzhang-microsoft
as code owners
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
YatishSVC
reviewed
Mar 30, 2026
Contributor
There was a problem hiding this comment.
Hi @venu-nexthop,
Are these changes intentional?
Original PR: https://github.com/sonic-net/sonic-mgmt/pull/22199/changes
Contributor
Author
There was a problem hiding this comment.
This happened because of some unintended change in our internal fork. Fixed now.
Signed-off-by: venu-nexthop <venu@nexthop.ai>
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
…gle asic voq duts" This reverts commit 0935ea9. Signed-off-by: venu-nexthop <venu@nexthop.ai>
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
venu-nexthop
force-pushed
the
venu-everflow-acl-rule-validate-202511
branch
from
f4959b4 to
0b0c0dc
Compare
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
AI agent on behalf of Ying.\n\nIssues:\n- Found TODO/FIXME in diff.\n\n |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
wait_untilfor ACL rules is now performed once at test startChanges
tests/everflow/everflow_test_utilities.py— Added ACL rule validation, VID/RID mapping check, multi-ASIC handlingtests/everflow/test_everflow_testbed.py— Added route and ACL rule verification before traffic testsTest plan
Original PR
#22199
🤖 Generated with Claude Code