-
Notifications
You must be signed in to change notification settings - Fork 5
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
CRAYSAT-1817: Automate procedure of setting next boot device to disk #240
CRAYSAT-1817: Automate procedure of setting next boot device to disk #240
Conversation
9ffb1e6
to
af71f4a
Compare
Tested on Rocket where disks are not represented in the format
|
dcedddd
to
4f868bb
Compare
4f868bb
to
e5562e0
Compare
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.
My comments are minor. Overall, this looks good.
e5562e0
to
bd06427
Compare
bd06427
to
72a5aef
Compare
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.
Looks good. Minor unit test improvement suggestions.
c1aa441
to
d0b81ac
Compare
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.
There are two unit tests of the same name, and one actually has a NameError
, but this is masked since only one of them (the one without the error) is executed.
My comments and suggestions should address this. Otherwise, this still looks good. Feel free to merge once you've addressed that.
d0b81ac
to
022369f
Compare
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.
looks good
IM:CRAYSAT-1817
Reviewer: Ryan
Summary and Scope
Automate the procedure of setting the next boot device to disk before the management nodes are powered off as part of the full-system shutdown.
Issues and Related PRs
CRAYSAT-1817
Testing
Tested on:
Rocket
Test description:
Will do full system shutdown and boot to see the behavior of boot order
Risks and Mitigations
Minimal
Pull Request Checklist