-
Notifications
You must be signed in to change notification settings - Fork 28
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
cnf ran: add ZTP verify BIOS settings test case #204
Conversation
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.
A few style comments. Is there supposed to be an actual configuring of BIOS settings as part of the test though? From what I heard from Yang it seemed like you would have to apply a change to the bios settings then verify the change showed up on the node
9e2542a
to
dfd649c
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.
/lgtm
/lgtm |
ZTP BIOS test added: Compare requested BIOS settings to current BIOS settings