Skip to content

[action] [PR:20175] Add Moby to policer unsupported list#20294

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/20175
Aug 18, 2025
Merged

[action] [PR:20175] Add Moby to policer unsupported list#20294
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/20175

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description of PR

Summary:
Fixes # (issue)
This is a follow-up PR of #19463
Remove the policer on the Moby platform due to hardware not supported when creating MIRROR_SESSION.
CS00012412189: [TH5-512]Error log "Platform does not support SAI_MIRROR_SESSION_ATTR_POLICER".

admin@str5-7060x6-moby-512-2:~$ show platform summary
Platform: x86_64-arista_7060x6_16pe_384c_b
HwSKU: Arista-7060X6-16PE-384C-B-O128S2
ASIC: broadcom
ASIC Count: 1
Serial Number: SSN25202343
Model Number: DCS-7060X6-16PE-384C-B
Hardware Revision: 03.00
admin@str5-7060x6-moby-512-2:~$ 

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202505

Approach

What is the motivation for this PR?

Add Moby to policer unsupported list when creating MIRROR_SESSION.

How did you do it?

Check the platform before building and verifying monitor session config, remove the policer if platform startwith "x86_64-arista_7060x6".

How did you verify/test it?

Manually run generic_config_updater/test_monitor_config.py on Moby testbed and case can pass.

generic_config_updater/test_monitor_config.py::test_monitor_config_tc1_suite[None] PASSED [100%]

Signed-off-by: zitingguo-ms [email protected]

What is the motivation for this PR?
Add Moby to policer unsupported list when creating MIRROR_SESSION.

How did you do it?
Check the platform before building and verifying monitor session config, remove the policer if platform startwith "x86_64-arista_7060x6".

How did you verify/test it?
Manually run generic_config_updater/test_monitor_config.py on Moby testbed and case can pass.

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

Original PR: #20175

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 615afbb into sonic-net:202505 Aug 18, 2025
12 checks passed
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.

2 participants