vpp: enable crm tests - #23316
Merged
Merged
Conversation
Signed-off-by: Yue Gao <yuega2@cisco.com>
Collaborator
|
/azp run |
github-actions
Bot
requested review from
wangxin,
xwjiang-ms and
yutongzhang-microsoft
March 25, 2026 17:14
|
Azure Pipelines successfully started running 1 pipeline(s). |
yue-fred-gao
marked this pull request as ready for review
March 30, 2026 18:27
Collaborator
|
AI agent on behalf of Ying. Issues found: test_ prefix |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
Author
|
/azpw run |
Collaborator
|
Retrying failed(or canceled) jobs... |
Collaborator
|
The following failed(or canceled) jobs were not retried:
Job kvmtest-t1-lag-vpp by Elastictest [OPTIONAL] is an optional job and does not block the PR merge, so it will not be retried. |
auspham
approved these changes
Mar 31, 2026
auspham
reviewed
Mar 31, 2026
auspham
left a comment
auspham
left a comment
Contributor
There was a problem hiding this comment.
https://elastictest.org/scheduler/testplan/69cbcbcb8d95038c79e91d4c test_crm still failing after enabled. Is this expected?
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
12 tasks
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Collaborator
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
rraghav-cisco
pushed a commit
to rraghav-cisco/sonic-mgmt
that referenced
this pull request
Apr 20, 2026
Description of PR Summary: Fixes # (issue) crm tests in vpp were disabled because sonic-vpp didn't support crm counters. With PR sonic-net/sonic-sairedis#1814 and sonic-net/sonic-sairedis#1811, the feature is enabled so we can turn on the test for sonic-vpp Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 202405 202411 202505 202511 Approach What is the motivation for this PR? enable crm/test_crm.py and crm/test_crm_available.py How did you do it? Remove sonic-vpp conditional marker and add the tests to pr_test_scripts for t1_lag_vpp How did you verify/test it? Run the corresponding tests in sonic-mgmt and all test passed Any platform specific information? vpp specific Signed-off-by: Yue Gao <yuega2@cisco.com> Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
rraghav-cisco
pushed a commit
to rraghav-cisco/sonic-mgmt
that referenced
this pull request
May 11, 2026
Description of PR Summary: Fixes # (issue) crm tests in vpp were disabled because sonic-vpp didn't support crm counters. With PR sonic-net/sonic-sairedis#1814 and sonic-net/sonic-sairedis#1811, the feature is enabled so we can turn on the test for sonic-vpp Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 202405 202411 202505 202511 Approach What is the motivation for this PR? enable crm/test_crm.py and crm/test_crm_available.py How did you do it? Remove sonic-vpp conditional marker and add the tests to pr_test_scripts for t1_lag_vpp How did you verify/test it? Run the corresponding tests in sonic-mgmt and all test passed Any platform specific information? vpp specific Signed-off-by: Yue Gao <yuega2@cisco.com> Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
selldinesh
pushed a commit
to selldinesh/sonic-mgmt
that referenced
this pull request
Jun 11, 2026
Description of PR Summary: Fixes # (issue) crm tests in vpp were disabled because sonic-vpp didn't support crm counters. With PR sonic-net/sonic-sairedis#1814 and sonic-net/sonic-sairedis#1811, the feature is enabled so we can turn on the test for sonic-vpp Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 202405 202411 202505 202511 Approach What is the motivation for this PR? enable crm/test_crm.py and crm/test_crm_available.py How did you do it? Remove sonic-vpp conditional marker and add the tests to pr_test_scripts for t1_lag_vpp How did you verify/test it? Run the corresponding tests in sonic-mgmt and all test passed Any platform specific information? vpp specific Signed-off-by: Yue Gao <yuega2@cisco.com> Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
selldinesh
pushed a commit
to selldinesh/sonic-mgmt
that referenced
this pull request
Jun 25, 2026
Description of PR Summary: Fixes # (issue) crm tests in vpp were disabled because sonic-vpp didn't support crm counters. With PR sonic-net/sonic-sairedis#1814 and sonic-net/sonic-sairedis#1811, the feature is enabled so we can turn on the test for sonic-vpp Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 202405 202411 202505 202511 Approach What is the motivation for this PR? enable crm/test_crm.py and crm/test_crm_available.py How did you do it? Remove sonic-vpp conditional marker and add the tests to pr_test_scripts for t1_lag_vpp How did you verify/test it? Run the corresponding tests in sonic-mgmt and all test passed Any platform specific information? vpp specific Signed-off-by: Yue Gao <yuega2@cisco.com> Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
ssithaia-ebay
pushed a commit
to ssithaia-ebay/sflow-yang-sonic-mgmt
that referenced
this pull request
Jul 21, 2026
Description of PR Summary: Fixes # (issue) crm tests in vpp were disabled because sonic-vpp didn't support crm counters. With PR sonic-net/sonic-sairedis#1814 and sonic-net/sonic-sairedis#1811, the feature is enabled so we can turn on the test for sonic-vpp Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 202405 202411 202505 202511 Approach What is the motivation for this PR? enable crm/test_crm.py and crm/test_crm_available.py How did you do it? Remove sonic-vpp conditional marker and add the tests to pr_test_scripts for t1_lag_vpp How did you verify/test it? Run the corresponding tests in sonic-mgmt and all test passed Any platform specific information? vpp specific Signed-off-by: Yue Gao <yuega2@cisco.com> Signed-off-by: ssithaia-ebay <ssithaian@ebay.com>
ytzur1
pushed a commit
to ytzur1/sonic-mgmt
that referenced
this pull request
Jul 22, 2026
Description of PR Summary: Fixes # (issue) crm tests in vpp were disabled because sonic-vpp didn't support crm counters. With PR sonic-net/sonic-sairedis#1814 and sonic-net/sonic-sairedis#1811, the feature is enabled so we can turn on the test for sonic-vpp Type of change Bug fix Testbed and Framework(new/improvement) New Test case Skipped for non-supported platforms Test case improvement Back port request 202205 202305 202311 202405 202411 202505 202511 Approach What is the motivation for this PR? enable crm/test_crm.py and crm/test_crm_available.py How did you do it? Remove sonic-vpp conditional marker and add the tests to pr_test_scripts for t1_lag_vpp How did you verify/test it? Run the corresponding tests in sonic-mgmt and all test passed Any platform specific information? vpp specific Signed-off-by: Yue Gao <yuega2@cisco.com> Signed-off-by: Yael Tzur <ytzur@nvidia.com>
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
5 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.
Description of PR
Summary:
Fixes # (issue)
crm tests in vpp were disabled because sonic-vpp didn't support crm counters. With PR sonic-net/sonic-sairedis#1814 and sonic-net/sonic-sairedis#1811, the feature is enabled so we can turn on the test for sonic-vpp
Type of change
Back port request
Approach
What is the motivation for this PR?
enable crm/test_crm.py and crm/test_crm_available.py
How did you do it?
Remove sonic-vpp conditional marker and add the tests to pr_test_scripts for t1_lag_vpp
How did you verify/test it?
Run the corresponding tests in sonic-mgmt and all test passed
Any platform specific information?
vpp specific
Supported testbed topology if it's a new test case?
Documentation