Skip to content

Conversation

@miabbott
Copy link
Member

We want to be clear that the coreos.inst.ignition_url and
coreos.inst.rootfs_url parameters only supports HTTP and HTTPS
when installing RHCOS via ISO/PXE.

We want to be clear that the `coreos.inst.ignition_url` and
`coreos.inst.rootfs_url` parameters only supports HTTP and HTTPS
when installing RHCOS via ISO/PXE.
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 30, 2020
@miabbott
Copy link
Member Author

This should be included as part of OCP 4.6 docs.

cc: @bgilbert @chrisnegus @bobfuru

Copy link
Contributor

@bgilbert bgilbert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM generally. One small fix.

bootstrap Ignition config file, and the `coreos.live.rootfs_url` parameter value is
the location of the live `rootfs` file.
the location of the live `rootfs` file. The `coreos.inst.ignition_url` and
`coreos.inst.rootfs_url` parameters only supports HTTP and HTTPS.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

coreos.live.rootfs_url (also in 3 other cases)

@bobfuru
Copy link
Contributor

bobfuru commented Oct 2, 2020

@JAORMX - Thanks for reviewing, Oz! FYI, we are completing this work in #26070, which was opened to address additional feedback from Jonathan.

@bobfuru
Copy link
Contributor

bobfuru commented Oct 2, 2020

Fixed in #26070 and #26093. Closing PR. Thanks, @miabbott!

@bobfuru bobfuru closed this Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants