Skip to content

Conversation

@hardys
Copy link

@hardys hardys commented Dec 18, 2019

Since openshift/installer#2757
merged we can mirror the installer referenced images directly
to simulate typical disconnected install scenarios.

This removes some complexity as we no longer need to run the
downloader container on the host, only expose the unprocessed images
via http.

Note this will work with latest 4.4. builds but not 4.3 until
openshift/installer#2835 lands

@hardys hardys requested a review from stbenjam December 18, 2019 15:20
@hardys
Copy link
Author

hardys commented Dec 18, 2019

/cc @kirankt FYI

@hardys hardys changed the title WIP Convert OS Image cache to use new install-config interface Convert OS Image cache to use new install-config interface Dec 18, 2019
@hardys
Copy link
Author

hardys commented Dec 19, 2019

coreos/coreos-assembler#1006 landed but we still need an RHCOS bump to avoid the "not in gzip format" error from rhcos-downloader on the virthost.

Merging this may be the quickest workaround for that, I'll fix up the MIRROR_IP and see if CI passes.

Since openshift/installer#2757
merged we can mirror the installer referenced images directly
to simulate typical disconnected install scenarios.

This removes some complexity as we no longer need to run the
downloader container on the host, only expose the unprocessed images
via http.

Note this will work with latest 4.4. builds but not 4.3 until
openshift/installer#2835 lands
@hardys hardys added the CI check this PR with CI label Dec 19, 2019
@hardys hardys requested a review from stbenjam December 19, 2019 10:26
@hardys hardys self-assigned this Dec 19, 2019
@metal3ci
Copy link

Build SUCCESS, see build http://10.8.144.11:8080/job/dev-tools/1388/

@stbenjam stbenjam merged commit c6383ca into openshift-metal3:master Dec 19, 2019
@hardys hardys deleted the rhcos_cache branch December 19, 2019 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI check this PR with CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants