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

Update bcm/config.bcm file for EC SAI on MV2 TD3 TH3 HX5 #215

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

peter-yu
Copy link

@peter-yu peter-yu commented Sep 23, 2024

Why I did it

Update the bcm configuration file for EC SAI

How I did it

  1. The file to be used with ecSAI will be named as 'xxx_ec.bcm' or 'xxx_ec.config.bcm'
  2. Accton-AS4630-54NPE didn't support community SAI. Update the bcm file directly.

How to verify it

N/A

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

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@peter-yu
Copy link
Author

@gord1306 @linarnan @chenkelly Please help to review

@peter-yu peter-yu marked this pull request as draft September 23, 2024 06:21
@peter-yu peter-yu force-pushed the pr-test2 branch 2 times, most recently from c807cb6 to 92a5e15 Compare September 23, 2024 06:40
@peter-yu peter-yu marked this pull request as ready for review September 23, 2024 06:42
@peter-yu peter-yu changed the title Sync bcm/config.bcm file from ec202211_ecsonic for HX5 Sync bcm/config.bcm file from ec202211_ecsonic for MV2 TD3 TH3 HX5 Sep 23, 2024
@gord1306
Copy link
Contributor

@peter-yu , please also update the Accton-AS9716-32D/th3-as9716-32x400G.config.bcm

@peter-yu
Copy link
Author

Note: This requires build script to replace xxx_ec.bcm to xxx.bcm

@peter-yu peter-yu marked this pull request as draft September 23, 2024 10:17
@peter-yu peter-yu changed the title Sync bcm/config.bcm file from ec202211_ecsonic for MV2 TD3 TH3 HX5 Update bcm/config.bcm file for ecSAI on MV2 TD3 TH3 HX5 Sep 24, 2024
@peter-yu peter-yu changed the title Update bcm/config.bcm file for ecSAI on MV2 TD3 TH3 HX5 Update bcm/config.bcm file for EC SAI on MV2 TD3 TH3 HX5 Sep 24, 2024
1. The file to be used with ecSAI will be named as 'xxx_ec.bcm'
2. Accton-AS4630-54NPE didn't support community SAI. Update the bcm file directly.
@peter-yu peter-yu force-pushed the pr-test2 branch 2 times, most recently from cd97b18 to e61c670 Compare September 24, 2024 09:35
@peter-yu peter-yu marked this pull request as ready for review September 24, 2024 09:37
@peter-yu
Copy link
Author

@gord1306 Please help to review

@peter-yu peter-yu marked this pull request as draft September 24, 2024 10:35
The file to be used with ecSAI will be named as xxx_ec.config.bcm
@peter-yu peter-yu marked this pull request as ready for review September 25, 2024 03:02
@peter-yu
Copy link
Author

@gord1306 Please help to review

@bryan1978 bryan1978 merged commit fbb0133 into edge-core:202311.X Sep 25, 2024
1 check passed
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