Skip to content

Set ENABLE to 0 for TH5 platform#23125

Merged
yxieca merged 1 commit intosonic-net:masterfrom
bingwang-ms:set_enable_0_th5
Jul 1, 2025
Merged

Set ENABLE to 0 for TH5 platform#23125
yxieca merged 1 commit intosonic-net:masterfrom
bingwang-ms:set_enable_0_th5

Conversation

@bingwang-ms
Copy link
Contributor

@bingwang-ms bingwang-ms commented Jun 27, 2025

Why I did it

Fixed https://github.com/aristanetworks/sonic-qual.msft/issues/645

Work item tracking
  • Microsoft ADO (number only):

How I did it

Update bcm config to set ENABLE to 0.

How to verify it

The change is verified by modifying the bcm config and checking if all ports can be up.

Which release branch to backport (provide reason below if selected)

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Tested branch (Please provide the tested image version)

  • 202503

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ccroy-arista
Copy link
Contributor

x86_64-arista_7060x6_64pe changes lgtm!
@dakotac-arista can you please review the x86_64-arista_7060x6_64pe_b changes?

@dakotac-arista
Copy link
Contributor

Changes to x86_64-arista_7060x6_64pe_b LGTM

@ccroy-arista
Copy link
Contributor

@bingwang-ms can we also backport to 202505?

Copy link
Collaborator

@davidm-arista davidm-arista left a comment

Choose a reason for hiding this comment

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

LGTM

@bingwang-ms
Copy link
Contributor Author

/azpw ms_conflict

@yxieca yxieca merged commit 978c84f into sonic-net:master Jul 1, 2025
20 checks passed
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202503: Azure/sonic-buildimage-msft#1308

@bingwang-ms
Copy link
Contributor Author

For 202505 branch, it has dependency on PR #22732

@r12f
Copy link
Contributor

r12f commented Jul 3, 2025

hi @bingwang-ms , do you mind to help with a manual cherry pick to 202412?

dakotac-arista pushed a commit to dakotac-arista/sonic-buildimage that referenced this pull request Jul 9, 2025
Why I did it
Fixed aristanetworks/sonic-qual.msft#645

How I did it
Update bcm config to set ENABLE to 0.

How to verify it
The change is verified by modifying the bcm config and checking if all ports can be up.
r12f added a commit to Azure/sonic-buildimage-msft that referenced this pull request Jul 9, 2025
Sync PR #23125 from sonic-net/sonic-buildimage to 202412.
Original PR: sonic-net/sonic-buildimage#23125

Co-authored-by: bingwang-ms <66248323+bingwang-ms@users.noreply.github.com>
@dakotac-arista
Copy link
Contributor

For 202505 branch, it has dependency on PR #22732

I have added it in #23264 which adds the dependency as well.

yejianquan pushed a commit that referenced this pull request Jul 11, 2025
* [Arista] Add HWSKU Arista-7060X6-64PE-B-P64 (#22734)

* Add qspf TH5-512 HWSKU Arista-7060X6-64PE-B-P64

* [Arista] Add HWSKU Arista-7060X6-64PE-B-P32O64 (#22732)

[Arista] Add HWSKU Arista-7060X6-64PE-B-P32O64

* Add stat_custom_receive0_management_mode to the Broadcom config file of TH5 platforms (#22825)

Add stat_custom_receive0_management_mode to the Broadcom config file of TH5 platforms

* [Arista][DT2] Add QoS to Arista-7060X6-64PE-B-P32O64 (#22836)

Why I did it
Add QoS settings from CSP to TH5-512 HWSKU Arista-7060X6-64PE-B-P32O64

How to verify it
Sanitized a fully wired DUT and confirmed all links came up.

* [Arista][DT2] Add QoS to Arista-7060X6-64PE-B-P64 (#22837)

Why I did it
Add QoS settings from CSP to TH5-512 HWSKU Arista-7060X6-64PE-B-P64

How to verify it
Sanitized a fully wired DUT and confirmed all links came up.

* Enable srv6 on TH5 (#23018)

Why I did it
Enable srv6 on TH5

* Set ENABLE to 0 for TH5 platform (#23125)

Why I did it
Fixed aristanetworks/sonic-qual.msft#645

How I did it
Update bcm config to set ENABLE to 0.

How to verify it
The change is verified by modifying the bcm config and checking if all ports can be up.

---------

Co-authored-by: kewei-arista <146126091+kewei-arista@users.noreply.github.com>
Co-authored-by: Dayou Liu <113053330+dayouliu1@users.noreply.github.com>
Co-authored-by: bingwang-ms <66248323+bingwang-ms@users.noreply.github.com>
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.

8 participants