Skip to content
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

workflows: build and push ignition-validate container from GH Actions #1336

Merged
merged 2 commits into from
Mar 23, 2022
Merged

workflows: build and push ignition-validate container from GH Actions #1336

merged 2 commits into from
Mar 23, 2022

Conversation

bgilbert
Copy link
Contributor

@bgilbert bgilbert commented Mar 20, 2022

Quay builds are amd64-only and haven't been especially reliable. Use GitHub Actions to build both amd64 and arm64 containers for the main branch and for tags, and push them to Quay. Continue building but not pushing containers on PR. Requires the QUAY_AUTH repo secret to be set to a Docker credential.

Ideally we would cross-build the arm64 container by having the Dockerfile specify FROM --platform=$BUILDPLATFORM for the builder container and set GOARCH=$TARGETARCH. However, Buildah < 1.24.1 doesn't support --platform in FROM. Build in emulation for now, and skip arm64 in PRs to speed up CI.

Fixes #1321.

@bgilbert
Copy link
Contributor Author

bgilbert commented Mar 20, 2022

Before landing this:

  • Get Quay robot account in coreos namespace
  • Add robot account credential to shared credential storage
  • Add QUAY_AUTH secret
  • Give robot account write access to Quay repo
  • Drop Quay build trigger and build failure notification

Reduce overhead by pulling in fewer deps.
Quay builds are amd64-only and haven't been especially reliable.  Use
GitHub Actions to build both amd64 and arm64 containers for the main
branch and for tags, and push them to Quay.  Continue building but not
pushing containers on PR.  Requires the QUAY_AUTH repo secret to be set
to a Docker credential.

Ideally we would cross-build the arm64 container by having the Dockerfile
specify FROM --platform=$BUILDPLATFORM for the builder container and set
GOARCH=$TARGETARCH.  However, Buildah < 1.24.1 doesn't support --platform
in FROM.  Build in emulation for now, and skip arm64 in PRs to speed up CI.

Fixes #1321.
Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

Nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build Quay container in GitHub Actions
2 participants