Skip to content

Fix for parse_show_copp_configuration for multi-asic dut#20946

Merged
rlhui merged 1 commit intosonic-net:masterfrom
sanjair-git:copp-masic
Nov 26, 2025
Merged

Fix for parse_show_copp_configuration for multi-asic dut#20946
rlhui merged 1 commit intosonic-net:masterfrom
sanjair-git:copp-masic

Conversation

@sanjair-git
Copy link
Copy Markdown
Contributor

@sanjair-git sanjair-git commented Oct 7, 2025

Description of PR

Summary:
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?

  • Verify if the DUT is multi_asic and modified commands based on the result.

How did you verify/test it?

  • Ran all the COPP test cases on T2 multi-asic DUT and made sure all the tests are passed.

Any platform specific information?

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

Documentation

image

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Oct 7, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Oct 7, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Oct 21, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Oct 21, 2025

Azure Pipelines successfully started running 1 pipeline(s).

abdosi
abdosi previously approved these changes Oct 29, 2025
View reviewed changes
@abdosi
Copy link
Copy Markdown
Contributor

abdosi commented Oct 29, 2025

Resolve conflict

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Nov 11, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Nov 11, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@sanjair-git
Copy link
Copy Markdown
Contributor Author

sanjair-git commented Nov 11, 2025

Resolve conflict

Done. Conflicts resolved.

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Nov 19, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Nov 19, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@rlhui rlhui merged commit 7294362 into sonic-net:master Nov 26, 2025
15 checks passed
vikumarks pushed a commit to vikumarks/sonic-mgmt that referenced this pull request Dec 1, 2025
@sanjair-git @vikumarks
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
4310e5f 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: vikumarks <vikumar7ks@gmail.com>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
@sanjair-git
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
0f7ffff 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
dcaugher pushed a commit to dcaugher/sonic-mgmt that referenced this pull request Dec 8, 2025
@sanjair-git @dcaugher
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
caed07b 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Dan Caugherty <dcaugher@cisco.com>
nissampa pushed a commit to nissampa/sonic-mgmt_dpu_test that referenced this pull request Dec 9, 2025
@sanjair-git @nissampa
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
581ad0b 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Nishanth Sampath Kumar <nissampa@cisco.com>
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Dec 11, 2025
@sanjair-git @selldinesh
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
fe3b7f7 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: selldinesh <dinesh.sellappan@keysight.com>
echuawu pushed a commit to echuawu/sonic-mgmt that referenced this pull request Dec 12, 2025
@sanjair-git @echuawu
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
9a5cb1d 
…onic-net#20946)

Revert is_trap_uninstalled method deletion
saravanan-nexthop pushed a commit to saravanan-nexthop/sonic-mgmt that referenced this pull request Dec 15, 2025
@sanjair-git @saravanan-nexthop
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
3c6daad 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Saravanan <saravanan@nexthop.ai>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 16, 2025
@sanjair-git @gshemesh2
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
71e0821 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
@sanjair-git @AharonMalkin
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
a030970 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
@sanjair-git @gshemesh2
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
f3262f7 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

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
@sanjair-git @venu-nexthop
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
94807f9 
…onic-net#20946)

Revert is_trap_uninstalled method deletion
yifan-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Jan 14, 2026
@sanjair-git @yifan-nexthop
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
84d8f43 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: YiFan Wang <yifan@nexthop.ai>
PriyanshTratiya pushed a commit to PriyanshTratiya/sonic-mgmt that referenced this pull request Jan 21, 2026
@sanjair-git @PriyanshTratiya
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
84c99ac 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Priyansh Tratiya <ptratiya@microsoft.com>
lakshmi-nexthop pushed a commit to lakshmi-nexthop/sonic-mgmt that referenced this pull request Jan 28, 2026
@sanjair-git @lakshmi-nexthop
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
8f877e0 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Lakshmi Yarramaneni <lakshmi@nexthop.ai>
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Jan 29, 2026
@sanjair-git @ytzur1
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
dfdef42 
…onic-net#20946)

Revert is_trap_uninstalled method deletion
ytzur1 pushed a commit to ytzur1/sonic-mgmt that referenced this pull request Feb 2, 2026
@sanjair-git @ytzur1
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
2d777b5 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

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
@sanjair-git @abhishek-nexthop
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
38f077f 
…onic-net#20946)

Revert is_trap_uninstalled method deletion
rraghav-cisco pushed a commit to rraghav-cisco/sonic-mgmt that referenced this pull request Feb 13, 2026
@sanjair-git @rraghav-cisco
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
c17a3d8 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Raghavendran Ramanathan <rraghav@cisco.com>
anilal-amd pushed a commit to anilal-amd/anilal-forked-sonic-mgmt that referenced this pull request Feb 19, 2026
@sanjair-git @ztan-amd
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
2248420 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Zhuohui Tan <zhuohui.tan@amd.com>
abhishek-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request Mar 17, 2026
@sanjair-git @abhishek-nexthop
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
c9b4599 
…onic-net#20946)

Revert is_trap_uninstalled method deletion

Signed-off-by: Abhishek <abhishek@nexthop.ai>
venu-nexthop pushed a commit to venu-nexthop/sonic-mgmt that referenced this pull request Mar 27, 2026
@sanjair-git
[GITPR_TBD] Fix for parse_show_copp_configuration for multi-asic dut (s…
60e06cd 
…onic-net#20946)

Revert is_trap_uninstalled method deletion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rlhui rlhui rlhui approved these changes

@abdosi abdosi abdosi left review comments

Assignees

No one assigned

Labels

None yet

Projects

SONiC Chassis
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sanjair-git @mssonicbld @abdosi @rlhui @mlok-nokia