Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Platform DCS-7060DX5-32 MMU rollback to avoid dependancy on Broadcom #21035

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

vincentpcng
Copy link
Contributor

driver patch

Why I did it

This commit is to avoid Broadcom driver dependency on the MMU config

Work item tracking
  • Microsoft ADO (number only):

How I did it

We upload the basic version of the Broadcom configure with limited configuration.

How to verify it

We verified the change internally in Arista

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

master and 202405

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

Tested branch (Please provide the tested image version)

202405 and master

  • master
  • 202405

Description for the changelog

Link to config_db schema for YANG module changes

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

driver patch

Signed-off-by: vincent ng <vincent.ng@arista.com>
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit 2b92c3d into sonic-net:master Dec 12, 2024
22 checks passed
@StormLiangMS
Copy link
Contributor

hi @bingwang-ms this is only impact 7060dx5, could you help to cherry pick?

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Dec 18, 2024
…onic-net#21035)

Why I did it
This commit is to avoid Broadcom driver dependency on the MMU config

Work item tracking
Microsoft ADO (number only):
How I did it
We upload the basic version of the Broadcom configure with limited configuration.

How to verify it
We verified the change internally in Arista
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202405: #21214

@byu343
Copy link
Contributor

byu343 commented Dec 18, 2024

We need this change to be backported to 202411. Thanks

@StormLiangMS
Copy link
Contributor

StormLiangMS commented Dec 19, 2024

hi @kperumalbfn could you help to cherry pick to 202411? This is for TH4.

mssonicbld pushed a commit that referenced this pull request Jan 4, 2025
…21035)

Why I did it
This commit is to avoid Broadcom driver dependency on the MMU config

Work item tracking
Microsoft ADO (number only):
How I did it
We upload the basic version of the Broadcom configure with limited configuration.

How to verify it
We verified the change internally in Arista
@StormLiangMS
Copy link
Contributor

Hi @kperumalbfn could you help to pick for 202411? It was a mistake that I thought it is not needed.

@kperumalbfn
Copy link
Contributor

@StormLiangMS Do we need this SKU to be added in GCU https://github.com/sonic-net/sonic-utilities generic_config_updater/gcu_field_operation_validators.conf.json?

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202411: #21438

VladimirKuk pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Jan 21, 2025
…onic-net#21035)

Why I did it
This commit is to avoid Broadcom driver dependency on the MMU config

Work item tracking
Microsoft ADO (number only):
How I did it
We upload the basic version of the Broadcom configure with limited configuration.

How to verify it
We verified the change internally in Arista
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.

6 participants