Skip to content

Increase timeout for test_crm.py::test_acl_counter#22672

Merged
StormLiangMS merged 3 commits intosonic-net:masterfrom
markx-arista:fix-test-crm-acl-counter-timeout
Mar 10, 2026
Merged

Increase timeout for test_crm.py::test_acl_counter#22672
StormLiangMS merged 3 commits intosonic-net:masterfrom
markx-arista:fix-test-crm-acl-counter-timeout

Conversation

@markx-arista
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
New wait and check mechanism was added in:
#22132
Arista 7050CX3-32C on m1-128 topo has been failing this test since then because it needs a slightly longer timeout
Fixes # (issue)

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?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit 9d18023 into sonic-net:master Mar 10, 2026
16 of 17 checks passed
@StormLiangMS StormLiangMS added the Request for 202511 branch Request to backport a change to 202511 branch label Mar 10, 2026
ksravani-hcl pushed a commit to ksravani-hcl/sonic-mgmt that referenced this pull request Mar 10, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
ksravani-hcl pushed a commit to ksravani-hcl/sonic-mgmt that referenced this pull request Mar 10, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
aronovic pushed a commit to aronovic/sonic-mgmt that referenced this pull request Mar 10, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
Signed-off-by: Mihut Aronovici <[email protected]>
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Mar 12, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
Signed-off-by: mssonicbld <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202511: #22900

selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Mar 16, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
Signed-off-by: selldinesh <[email protected]>
mssonicbld pushed a commit that referenced this pull request Mar 17, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
Signed-off-by: mssonicbld <[email protected]>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
Signed-off-by: Abhishek <[email protected]>
vrajeshe pushed a commit to vrajeshe/sonic-mgmt that referenced this pull request Mar 23, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
Signed-off-by: Venkata Gouri Rajesh Etla <[email protected]>
ravaliyel pushed a commit to ravaliyel/sonic-mgmt that referenced this pull request Mar 27, 2026
Some platforms need a slightly longer timeout for ACL counter resources

Signed-off-by: markxiao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants