Skip to content

[Arista]: Add HWSKU Arista-7060X6-64PE-B-O128S2#22610

Merged
yxieca merged 5 commits intosonic-net:masterfrom
dakotac-arista:master-add-Arista-7060X6-64PE-B-O128S2
Jun 12, 2025
Merged

[Arista]: Add HWSKU Arista-7060X6-64PE-B-O128S2#22610
yxieca merged 5 commits intosonic-net:masterfrom
dakotac-arista:master-add-Arista-7060X6-64PE-B-O128S2

Conversation

@dakotac-arista
Copy link
Contributor

@dakotac-arista dakotac-arista commented May 14, 2025

Why I did it

Added HWSKU Arista-7060X6-64PE-B-O128S2 for new deployment of 128 x 400g, 2 x 10g ports on TH-512.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Tested hwsku on dut and all interfaces worked.
ls -al shows expected path for symlink and also files within original and symlink directory.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305
  • msft-202503

Tested branch (Please provide the tested image version)

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

@StormLiangMS StormLiangMS requested review from r12f and sdszhang May 14, 2025 07:50
@StormLiangMS
Copy link
Contributor

hi @r12f @sdszhang could you help to take a look?

Copy link
Contributor

@sdszhang sdszhang left a comment

Choose a reason for hiding this comment

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

is buffer configuration going to be included in this PR?

@dakotac-arista
Copy link
Contributor Author

is buffer configuration going to be included in this PR?

We are currently waiting on information from Broadcom. The qos and buffer files will be added in a follow up PR.

@StormLiangMS StormLiangMS requested a review from bingwang-ms May 15, 2025 04:58
@dakotac-arista
Copy link
Contributor Author

Hi @bingwang-ms, Should I make symlinks for non S2 variants or should I make new HWSKUs?

@bingwang-ms
Copy link
Contributor

Buffer configuration is missing from the HWSKU. Is that expected?

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Contributor

Hi @bingwang-ms, Should I make symlinks for non S2 variants or should I make new HWSKUs?

There is another PR for same HWSKU but w/o S2 https://github.com/sonic-net/sonic-buildimage/pull/22672/files

@dakotac-arista
Copy link
Contributor Author

Hi @bingwang-ms, Should I make symlinks for non S2 variants or should I make new HWSKUs?

There is another PR for same HWSKU but w/o S2 https://github.com/sonic-net/sonic-buildimage/pull/22672/files

Yes, I removed the symlink for O128 because it will be handled in that PR. This PR will only be for the S2 version.

I added default buffer configurations for LT2. From my understanding this HWSKU will be used as a fanout and doesn't need QoS. Do I need to add anything else or make any changes?

@bingwang-ms
Copy link
Contributor

Hi @bingwang-ms, Should I make symlinks for non S2 variants or should I make new HWSKUs?

There is another PR for same HWSKU but w/o S2 https://github.com/sonic-net/sonic-buildimage/pull/22672/files

Yes, I removed the symlink for O128 because it will be handled in that PR. This PR will only be for the S2 version.

I added default buffer configurations for LT2. From my understanding this HWSKU will be used as a fanout and doesn't need QoS. Do I need to add anything else or make any changes?

For Arista-7060X6-64PE-B-O128S2, I would like @r12f to comment.
For Arista-7060X6-64PE-O128 in PR #22672, it will be used in prod. So QoS is a must.

@sdszhang
Copy link
Contributor

sdszhang commented Jun 3, 2025

pls include the change in #22825 for this hwsku too.

@kewei-arista
Copy link
Contributor

pls include the change in #22825 for this hwsku too.

I'll update my RP after this PR merges.

@dakotac-arista
Copy link
Contributor Author

@sdszhang I've added the fix so #22825 isn't blocked. I'm waiting on a comment from @r12f for next steps on this HWSKU.

@r12f
Copy link
Contributor

r12f commented Jun 9, 2025

/azp run Azure.sonic-buidlimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 22610 in repo sonic-net/sonic-buildimage

@r12f
Copy link
Contributor

r12f commented Jun 9, 2025

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 22610 in repo sonic-net/sonic-buildimage

@StormLiangMS
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit fe50bd3 into sonic-net:master Jun 12, 2025
17 checks passed
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #22966

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to msft-202412: Azure/sonic-buildimage-msft#1240

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.

10 participants