Add support for the Celestica Redstone XP D2060 #11426
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
|
This pull request introduces 8 alerts when merging 82afe468ddb0bde84cb384cb0a9bf3885629b012 into 503a0f7 - view on LGTM.com new alerts:
|
device/celestica/x86_64-cel_redstone_xp-r0: Added device configuration and sonic_platform. * Added sonic device configurations. * Added SAI configuration. * Added LED code. * Added "legacy" plugins until they are deprecated. * Added sonic_platform. platform/broadcom: Added entries to package build list. platform/broadcom/sonic-platform-cel: Added gitignore to add debian folder. platform/broadcom/sonic-platform-cel/debian: Added debian package configuration and init/install scripts. platform/broadcom/sonic-platform-cel/redstonexp: Added platform drivers. * Added platform drivers. * Added platform config. Signed-off-by: Derek Vance DACepair@gmail.com
|
This pull request introduces 8 alerts when merging d551fc9 into 429254c - view on LGTM.com new alerts:
|
device/celestrica/x86_64-cel_redstone_xp-r0/sonic_platform/helper.py: `BaseException` instances handled. device/celestrica/x86_64-cel_redstone_xp-r0/sonic_platform/psu.py: Changed `Fan` class to use extended one. device/celestrica/x86_64-cel_redstone_xp-r0/sonic_platform/thermal_manager.py: Fixed polluting imports. platform/broadcom/sonic-platform-cel/redstonexp/setup.py: Removed `import os`.
|
This pull request introduces 1 alert when merging f9af481 into 429254c - view on LGTM.com new alerts:
|
…nged use of the `Fan` class for the PSU fans. device\celestica\x86_64-cel_redstone_xp-r0\sonic_platform\fan.py: Changed the logic to calculate if the fan is part of a PSU.
|
This pull request introduces 1 alert when merging 83bf8ce into bec35df - view on LGTM.com new alerts:
|
|
I'd love seeing the Redstone XP supported by SONiC, why is this PR stuck ? |
|
How could we get this to work? If someone redeveloped this against current master branch? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why I did it
There have been requests for something to run on this switch by the homelab community. I ordered one myself before realizing that there was practically no support. After many sleepless nights, and learning how interesting the platform was I decided to give it a whirl.
How I did it
The Celestica DX010 and Redstone XP use the same base platform. Much of the documentation can be found within the DX010 code, as well as through debugging with Cumulus and ONL.
How to verify it
The platform uses the Broadcom implementation of SONiC. Building the image using
PLATFORM=broadcomwill produce a usable image on this platform.Which release branch to backport (provide reason below if selected)
Description for the changelog
Added support for the Celestica Redstone XP D2060
Link to config_db schema for YANG module changes
This does not include any additional YANG changes.
A picture of a cute animal (not mandatory but encouraged)
Here's my pupper:
