Skip to content

[Arista] Add MMU configs for Arista-7050CX3-32S-C28S4#22023

Closed
byu343 wants to merge 1 commit intosonic-net:masterfrom
byu343:mmu-c28s4
Closed

[Arista] Add MMU configs for Arista-7050CX3-32S-C28S4#22023
byu343 wants to merge 1 commit intosonic-net:masterfrom
byu343:mmu-c28s4

Conversation

@byu343
Copy link
Contributor

@byu343 byu343 commented Mar 13, 2025

Why I did it

Add MMU configs for Arista-7050CX3-32S-C28S4

Work item tracking
  • Microsoft ADO (number only):

How I did it

Original values are from Arista-7050CX3-32S-C32
pg_profile_lookup.ini is revised for 10G by copying values from other speed

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

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)

Original values are from Arista-7050CX3-32S-C32
pg_profile_lookup.ini is revised for 10G by copying values from other speed
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@@ -0,0 +1,52 @@
{%- set default_cable = '300m' %}
Copy link
Contributor

Choose a reason for hiding this comment

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

this PR looks the same https://github.com/sonic-net/sonic-buildimage/pull/22117/files. Not sure if I missed anything.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i was not aware of the other change. Then this one could be closed. Thanks.

@byu343
Copy link
Contributor Author

byu343 commented Apr 2, 2025

The issue has been addressed by #22117

@byu343 byu343 closed this Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants