-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: Adds ironic IPA image build tools #308
Conversation
4b4b393
to
ca906a9
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.
Definitely think we need to drop the hardcoded auth and use it dynamically if we need it.
976cbd9
to
c7cef30
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.
two minor suggestions, but otherwise lgtm
ironic-images/custom_elements/undercloud-ipa/install.d/50-PUC-427
Outdated
Show resolved
Hide resolved
859244e
to
69ec1a1
Compare
69ec1a1
to
4054f8f
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.
So I'll merge this but I don't think we should be publishing a release on a PR. A build is fine.
No description provided.