-
Notifications
You must be signed in to change notification settings - Fork 756
[SmartSwitch] Extend reboot script for rebooting SmartSwitch #3566
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
Merged
Merged
Changes from 50 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
1686dbe
Extend reboot script for rebooting SmartSwitch
vvolam 23461b2
Add more coverage
vvolam cef5de7
Add more unittests and optimize tests file
vvolam d41bf43
Fix minor indentation
vvolam 68e70ab
Move smartswitch helper functions to new reboot_smartswitch_helper.sh
vvolam 3848b75
Fix pre-commit errors
vvolam 84d9e50
Fix few more indentation errors
vvolam ba5cd5d
Merge remote-tracking branch 'origin/master' into ss-reboot
vvolam 7f75134
Merge remote-tracking branch 'origin/master' into ss-reboot
vvolam a849e41
Add a new API in chassis.py
vvolam 4975ac0
Fix issues while testing
vvolam bead103
Fix indentation errors
vvolam f88491a
Add DPU_BUS_INFO
vvolam b3dbc0f
Fix pre-commit errors
vvolam 2d8b908
Add more error handling scenarios and increase more coverage
vvolam 1a6ef04
parse_args function is not required
vvolam ec21d6f
Fix indentation
vvolam 8d59222
Address review comments
vvolam 98406c7
Increase code coverage
vvolam a6f771e
Update scripts/reboot_smartswitch_helper
vvolam a3f8af7
Update scripts/reboot_smartswitch_helper
vvolam 36ecf1b
Rename module_base.py to module.py
vvolam 67e7817
Committing missed files in previous commit
vvolam 88af21d
Define a new try_get_args() which takes arguments as inputs
vvolam 3551a5a
Merge remote-tracking branch 'origin/master' into ss-reboot
vvolam e4bcc95
Fix some arguments
vvolam edaa0de
Exit the reboot script after completing DPU reboot
vvolam 119d83b
Fix long lines
vvolam 03fd56f
Merge remote-tracking branch 'origin/master' into ss-reboot
vvolam 75f4c26
Update unit tests for update function code
vvolam 7b365fd
Merge remote-tracking branch 'public/master' into ss-reboot
vvolam 2690348
Update scripts/reboot
vvolam dc504ea
Address few review comments
vvolam e878277
Merge remote-tracking branch 'public/master' into ss-reboot
vvolam d1488e8
Merge remote-tracking branch 'public/master' into ss-reboot
vvolam 1aa56e1
Add -notls option to gnoi_client command
vvolam 9aa2c88
Small fix to read smart_switch correctly
vvolam 2d7d141
Merge remote-tracking branch 'origin/master' into ss-reboot
vvolam 416fa37
Call the module in upper case
vvolam 4f0239e
Merge remote-tracking branch 'origin/master' into ss-reboot
vvolam af899a3
Use the string to HALT instead of numeric
vvolam d8bf907
Suppress outputs
vvolam 61dcbb3
Few enhancements to error path
vvolam f5c38e5
insecure communication is not required to GNMI server
vvolam 9f1b9e1
Enhance logging for better troubleshooting
vvolam 91e2e04
Small fix to not split the line
vvolam 02ccc84
Default port of GNMI server is 8080, in the absence of configuration
vvolam 635cdc4
Merge remote-tracking branch 'public/master' into ss-reboot
vvolam 0bec6c8
Remove unneccessary checks
vvolam d9c6e33
Small fix to fix gnoi_client command line
vvolam 21d84ec
Address comments
vvolam c7b4ab5
Merge remote-tracking branch 'public/master' into ss-reboot
vvolam 389ae8a
Minor fixes according to latest GNOI backend changes
vvolam 73300df
Wait for DPU reboot status only if reboot gnoi command is successful
vvolam 2cdf0c9
Skip reboot if DPU is not operationally online
vvolam d22356e
Fix oper status check
vvolam File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.