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
Bug 1808456: OpenStack: allow to overwrite default os image in the install config #3170
Bug 1808456: OpenStack: allow to overwrite default os image in the install config #3170
Conversation
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sdodson 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 |
/test e2e-libvirt |
docs/user/openstack/customization.md
Outdated
|
||
**NOTE:** For this to work, the parameter value must be a valid http(s) URL. | ||
|
||
**NOTE:** The optional `sha256` query parameter can be attach to the URL, which will force the installer to check the image file checksum before uploading it into Glance. |
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.
s/attach/attached/
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.
done
docs/user/openstack/customization.md
Outdated
@@ -90,6 +91,37 @@ pullSecret: '{"auths": ...}' | |||
sshKey: ssh-ed25519 AAAA... | |||
``` | |||
|
|||
## Image Overrides | |||
|
|||
When doing a disconnected installation, the OpenStack platform has the additional requirement that we have locations to download the RHCOS images. The installer downloads these from a location described in |
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.
When doing a disconnected installation
strictly speaking, that requirement is always present, and not only for disconnected installs.
What about something like:
The installer downloads the RHCOS image from a predetermined location (cf [data/data/rhcos.json](/data/data/rhcos.json)). The download URL can be overridden, notably for disconnected installations.
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.
done
docs/user/openstack/customization.md
Outdated
clusterOSImage: my-rhcos | ||
``` | ||
|
||
**NOTE:** The only difference in behavior with the previous example is that the value here is not an http(s) 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.
It's not the only difference! The sha256 checksum is missing as well. I'd say instead:
If `clusterOSImage` is not an URL, then the installer will look into Glance for an image with that name.
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.
done
1dc40ef
to
a64890d
Compare
/lgtm 🎉 |
/hold |
/retest |
@Fedosin: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/hold cancel |
@Fedosin: All pull requests linked via external trackers have merged. Bugzilla bug 1808456 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/cherry-pick release-4.3 |
@Fedosin: #3170 failed to apply on top of branch "release-4.3":
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/cherry-pick release-4.4 |
@Fedosin: new pull request created: #3205 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
This PR adds
clusterOSImage
parameter to the install config, that allows to override default OS image location for users.