-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[Seastone] Add new HWSKU for 48x50G+8x40G #4156
Conversation
Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
8701f8e
to
7d1a649
Compare
@zhenggen-xu, what is 50-50-40 means? It doesn't look to convey the meaning.. Can you follow the name like D48Q8 (Refer https://github.com/Azure/sonic-buildimage/tree/master/device/arista/x86_64-arista_7060_cx32s/Arista-7060CX-32S-D48C8) Also you've https://github.com/Azure/sonic-buildimage/tree/master/device/celestica/x86_64-cel_seastone-r0/Seastone-DX010-50-40 and the only change on this is the number of 50G ports |
retest vsimage please |
The HWSKU names are used internally and already deployed so won't be able to change. I was not aware of any naming convention of the HWSKU. To understand the meaning, people who deployment it just look at the port_config.ini. For future HWSKUs I would follow the convention if any. To be honest, I personally don't know what 32S-D48C8 means either. :-) |
Add the port_config.ini and sai.profile, leveraging the same bcm file. Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
Add the port_config.ini and sai.profile, leveraging the same bcm file. Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
[Seastone] Add new HWSKU for 48x50G+8x40G
Signed-off-by: Zhenggen Xu zxu@linkedin.com
Need backport to 201911.
- What I did
[Seastone] Add new HWSKU for 48x50G+8x40G
- How I did it
Add the port_config.ini and sai.profile, leveraging the same bcm file.
- How to verify it
Build image and config to new HWSKU:
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)