-
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
Commit the Silverstone platform of Celestica to Azure #3474
Conversation
update master branch
code sync
device/celestica/x86_64-cel_silverstone-r0/Silverstone-128x100/minigraph.xml
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,854 @@ | |||
<DeviceMiniGraph xmlns:i="http://www.w3.org/2001/XMLSchema-instance" xmlns="Microsoft.Search.Autopilot.Evolution"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no minigraph, use default sku, can you check the latest porting guide.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will remove Minigraph
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed in bf1a418.
Please share the output for below CLIs: -- show environment -- show interface transceiver eeprom On each HWSKU: |
device/celestica/x86_64-cel_silverstone-r0/Silverstone/port_config.ini
Outdated
Show resolved
Hide resolved
device/celestica/x86_64-cel_silverstone-r0/Silverstone/port_config.ini
Outdated
Show resolved
Hide resolved
device/celestica/x86_64-cel_silverstone-r0/Silverstone-128x100/minigraph.xml
Outdated
Show resolved
Hide resolved
SfpUtilBase.__init__(self) | ||
|
||
def get_presence(self, port_num): | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this blank line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this blank line
This blank line will be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed in a2d7757.
device/celestica/x86_64-cel_silverstone-r0/th3-128x100G.config.bcm
Outdated
Show resolved
Hide resolved
*These ports will be added later after BRCM SAI support SFP.
HwSKU-Silverstone.log The attachment is for CLI command log. |
-- "show environment" output does not seem to be correct. We should get the fan speed, sensor temperature etc. I assume that is probably from BMC, which means we need someway to get it from there? -- We need understand why the "show interface transceiver eeprom -d" is failing. |
About "show environment", I will check with plarform/BMC engineer. Anyway, We will check it. |
Build finished. No test results found. |
A note, "show environment" issue will be addressed in a different PR. This PR is approved itself. |
- What I did
Commit the Silverstone platform to Azure
- How I did it
Add 32x400G and 128x100G configuration for Silverstone platform
- How to verify it
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)