Skip to content

Stabilize the copp test case#20595

Merged
roy-sror merged 1 commit intosonic-net:masterfrom
nhe-NV:stablize_copp_test
Oct 27, 2025
Merged

Stabilize the copp test case#20595
roy-sror merged 1 commit intosonic-net:masterfrom
nhe-NV:stablize_copp_test

Conversation

@nhe-NV
Copy link
Copy Markdown
Contributor

@nhe-NV nhe-NV commented Sep 10, 2025

The new check point in the test_remove_trap was added by the community recently, but it failed on the SN2700 A0 from time to time, check few more time to make sure the test stable.

Description of PR

Summary: The new checker added in the PR: #18326, and for some platform which has weak CPU, it will take more take to apply the config, enhance the test case to make it stable.
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

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

@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).

@nhe-NV
Copy link
Copy Markdown
Contributor Author

nhe-NV commented Sep 13, 2025

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@nhe-NV
Copy link
Copy Markdown
Contributor Author

nhe-NV commented Sep 15, 2025

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines 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).

The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@nhe-NV
Copy link
Copy Markdown
Contributor Author

nhe-NV commented Oct 13, 2025

@roy-sror can you help to review?

@nhe-NV
Copy link
Copy Markdown
Contributor Author

nhe-NV commented Oct 13, 2025

@rminnikanti can you please to review?

Copy link
Copy Markdown
Contributor

@rminnikanti rminnikanti left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@roy-sror roy-sror merged commit a2cdc8e into sonic-net:master Oct 27, 2025
14 checks passed
dcaugher pushed a commit to dcaugher/sonic-mgmt that referenced this pull request Nov 12, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
dcaugher pushed a commit to dcaugher/sonic-mgmt that referenced this pull request Nov 12, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
vikumarks pushed a commit to vikumarks/sonic-mgmt that referenced this pull request Dec 1, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: vikumarks <vikumar7ks@gmail.com>
albertovillarreal-keys pushed a commit to albertovillarreal-keys/sonic-mgmt that referenced this pull request Dec 2, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Alberto Villarreal <alberto.villarreal@keysight.com>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Dec 11, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
echuawu pushed a commit to echuawu/sonic-mgmt that referenced this pull request Dec 12, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
saravanan-nexthop pushed a commit to saravanan-nexthop/sonic-mgmt that referenced this pull request Dec 15, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Saravanan <saravanan@nexthop.ai>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Jan 13, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
yifan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 14, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: YiFan Wang <yifan@nexthop.ai>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Jan 28, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Lakshmi Yarramaneni <lakshmi@nexthop.ai>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Jan 29, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Yael Tzur <ytzur@nvidia.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Feb 6, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.

Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
The new check point in the test_remove_trap was added by the community
recently, but it failed on the SN2700 A0 from time to time, check few
more time to make sure the test stable.
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