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
Change endpoint URL recording for ironic 15.x #85
Change endpoint URL recording for ironic 15.x #85
Conversation
We seem to have a version of this in the upstream image, I guess we can't just cherry-pick it as it also removes api_url ? |
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.
api_url
was deprecated back in Queens, we can safely switch to endpoint_override
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.
This change should be a backport of metal3-io@e1cb406
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.
let's do as suggested by @derekhiggins and @dtantsur and cherry pick the upstream change ?
api_url has been deprecated, changing it for endpoint_override
50afc33
to
c75bf76
Compare
/lgtm |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dtantsur, elfosardo, hardys, juliakreger The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Disable listening for DNS
Ironic 15.x removes a long deprecated [conductor]api_url
parameter. This patch prepares for that by setting the correct
parameter, and notes in-line the old parameter that can be
eventually removed once OpenShift begins consuming ironic 15.x.