-
Notifications
You must be signed in to change notification settings - Fork 159
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
bootupd-validate: remove condition as we have newer bootupd #3300
Conversation
Hold on, as not yet tested it on ppc64le, will do testing if I could setup the env. |
c564505
to
c850204
Compare
Test on ppc64le using
Test on ppc64le using
cosa build c9s, and run test, result is PASS.
|
considering openshift/os#1687 (comment) do we still need this? |
c850204
to
1d78f61
Compare
Thanks for the pointer, refresh the code to remove the condition as we have the newer bootupd (https://issues.redhat.com/browse/RHEL-71040), could you help to review, 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 but suggested some better wording
1d78f61
to
ddc9f3b
Compare
Fixes openshift/os#1687