-
Notifications
You must be signed in to change notification settings - Fork 166
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 for the procedures for insertion/hot swap of Switch Fabric Module(SFM) by using "config chassis modules shutdown/startup" commands #475
Conversation
…ule(SFM) by using "config chassis modules shutdown/startup" commands
can we add Design doc as discussed in the Chassis Community meeting ? |
@bmridul for review. |
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.
It looks good to me
@arlakshm @judyjoseph Please help to review this PR, tahnsk |
In Unit test, the operational status of Empty does not seem correct. The operational status could be "Offline" or some other word. However if the card has not been taken out of the chassis then it should not be Empty. Empty should indicate if the card has been physically removed from the chassis. $ show chassis modules status ... |
@mlok-nokia, please help to comment on this.
|
On nokia platform, When a SFM is power off, it cannot be detected even it is inserted in the chassis. Therefore, it is treated as empty instead of offline. |
|
@judyjoseph , please review it again and approve it for merging. Thanks. |
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.
LGTM
This PR has been replaced by #491. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@JunhongMao Please close this PR since it is no longer valid. |
Why I did it
For the Nokia SONiC chassis procedures for insertion/hot swap of Switch Fabric Module(SFM),
the previous solution was using the below commands.
The below 4 PRs intend to add the below commands for the equivalent operations.
#475
sonic-net/sonic-utilities#3283
nokia/sonic-platform#6
sonic-net/sonic-buildimage#18938
The HLD for Shutdown and Startup of the Fabric Module is below:
sonic-net/SONiC#1694
The below PR was replaced.
sonic-net/sonic-buildimage#18578
Work item tracking
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)