Skip to content
This repository was archived by the owner on Jul 23, 2019. It is now read-only.

Add a Dockerfile for building an installer binary image#53

Merged
russellb merged 1 commit intoopenshift-metal3:masterfrom
markmc:image-build
Apr 24, 2019
Merged

Add a Dockerfile for building an installer binary image#53
russellb merged 1 commit intoopenshift-metal3:masterfrom
markmc:image-build

Conversation

@markmc
Copy link
Copy Markdown
Contributor

@markmc markmc commented Apr 22, 2019

This is just a variant of images/installer/Dockerfile.ci that builds an image with kni-install at /usr/bin/openshift-install - this is so oc adm release extract --tools knows how to extact it.

We also install libvirt-libs in the resulting image since the openshift-install binary is dynamically linked when using libvirt because it's built withCGO_ENABLED=1 and there is no static libvirt library shipped in libvirt RPM builds.

Eventually, this dynamically linked binary will probably need to be published as a separate image (or perhaps in installer-artifacts) similarly to how the binary for mac is made available.

See also openshift-metal3/dev-scripts#401

This is just a variant of images/installer/Dockerfile.ci that builds
an image with kni-install at /usr/bin/openshift-install - this is so
'oc adm release extract --tools' knows how to extact it.

We also install libvirt-libs in the resulting image since the
openshift-install binary is dynamically linked when using libvirt
because it's built with CGO_ENABLED=1 and there is no static libvirt
library shipped in libvirt RPM builds.

Eventually, this dynamically linked binary will probably need to be
published as a separate image (or perhaps in installer-artifacts)
similarly to how the binary for mac is made available.
Comment thread images/baremetal/Dockerfile.ci
@russellb russellb merged commit c41e445 into openshift-metal3:master Apr 24, 2019
@markmc markmc deleted the image-build branch July 8, 2019 17:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants