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

[DellEMC:Z9332f] Modify SKU-DellEMC-Z9332f-M-O16C64 to support different port-config #6906

Merged
merged 2 commits into from
Mar 1, 2021

Conversation

srideepDell
Copy link
Contributor

@srideepDell srideepDell commented Feb 26, 2021

Why I did it

Modified DellEMC-Z9332f-M-O16C64 SKU as per new requirement change in portconfig.

How I did it

Change port_config.ini file with new port map

How to verify it

Verified on Z9332 switch logs attached.

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

2012 branch support Z9332.

  • [X ] 202012

Description for the changelog

Modified DellEMC-Z9332f-M-O16C64 SKU as per new requirement change in portconfig

@srideepDell srideepDell marked this pull request as ready for review February 26, 2021 09:34
@srideepDell
Copy link
Contributor Author

srideepDell commented Feb 26, 2021

Log attached.
Z9332_sku_onboarding.txt

@daall
Copy link
Contributor

daall commented Feb 28, 2021

Just to confirm @srideepDell @jeff-yin, does this require any changes in the config.bcm file, or just the port_config file?

@srideepDell
Copy link
Contributor Author

Just to confirm @srideepDell @jeff-yin, does this require any changes in the config.bcm file, or just the port_config file?

Just changes to port_config is good enough and it is tested. config.bcm already has the changes.

@daall daall merged commit 0f1d41d into sonic-net:master Mar 1, 2021
lguohan pushed a commit that referenced this pull request Mar 4, 2021
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
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.

2 participants