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

Improve & rename main build-push script #300

Merged
merged 3 commits into from
Sep 18, 2023

Conversation

cevich
Copy link
Member

@cevich cevich commented Sep 14, 2023

Revert 874da1b

This script orchestrates running of the actual build_push.sh script,
on behalf of various github containers-org repos. Rename it to better
reflect that purpose.

Change behavior WRT first argument (git repo. URL) to shallow-clone the
repo into a temporary directory.

Remove the auto-update library in anticipation of executing builds from
Cirrus-cron in this (automation_images) repo. Given encrypted secrets
are protected by execution context and actor.

Update labeling to also annotate the images, since newer tooling prefers
annotations but older tools only support labels.

@cevich cevich added no_skopeo_cidev Don't build the skopeo_cidev image no_prior-fedora Don't build any prior-fedora images no_fedora Don't build any fedora images no_fedora-aws Don't build any fedora-aws images no_fedora-aws-arm64 Don't build any fedora-aws-arm64 images no_fedora_podman Don't build the fedora_podman image no_prior-fedora_podman Don't build the prior-fedora_podman image no_fedora-netavark Don't build the fedora-netavark image no_fedora-netavark-aws-arm64 Don't build the fedora-netavark-aws image no_fedora-podman-py Don't build the fedora-podman-py image no_image-builder Don't build the image-builder image no_fedora-podman-aws-arm64 Don't build the fedora-podman-aws image no_win-server-wsl Don't build Windows WSL image no_debian Don't build the debian image no_rawhide Don't build the rawhide image and removed no_fedora Don't build any fedora images labels Sep 14, 2023
@cevich cevich force-pushed the multiarch_mulligan branch 2 times, most recently from bf636cb to 3e9a42b Compare September 14, 2023 20:13
Removed out of an abundance of caution, ref:
containers/podman#19796

Double-checked Cirrus-CI 'Decrypt Credentials' setting for this repo.
is: Collaborators, Bots, and Users with Write permission.

Double-checked Github collaboration settings.  It's limited to specific
github users only.

Signed-off-by: Chris Evich <cevich@redhat.com>
@cevich cevich force-pushed the multiarch_mulligan branch 11 times, most recently from 546e73c to 9bea7d6 Compare September 15, 2023 20:43
@cevich cevich force-pushed the multiarch_mulligan branch 2 times, most recently from dc02d1a to 99aff28 Compare September 18, 2023 14:59
This reverts commit 874da1b.

Signed-off-by: Chris Evich <cevich@redhat.com>
@cevich cevich force-pushed the multiarch_mulligan branch 2 times, most recently from bf6acef to 014ff0a Compare September 18, 2023 15:50
@cevich cevich changed the title [WIP] Improve & rename main build-push script Improve & rename main build-push script Sep 18, 2023
@cevich cevich force-pushed the multiarch_mulligan branch 2 times, most recently from 37e3270 to 31260c1 Compare September 18, 2023 16:10
@github-actions
Copy link

Cirrus CI build successful. Found built image names and IDs:

Stage Image Name IMAGE_SUFFIX
base fedora do-not-use
cache build-push c20230918t160949z-f38f37d13
cache fedora c20230918t160949z-f38f37d13

This script orchestrates running of the actual `build_push.sh` script,
on behalf of various github containers-org repos.  Rename it to better
reflect that purpose.

Change behavior WRT first argument (git repo. URL) to shallow-clone the
repo into a temporary directory.

Remove the auto-update library in anticipation of executing builds from
Cirrus-cron in this (automation_images) repo.  Given encrypted secrets
are protected by execution context and actor.

Update labeling to also annotate the images, since newer tooling prefers
annotations but older tools only support labels.

Remove wait-for-copr from build-push VM image since it's not needed.  An
alternate build system was put in place.

Signed-off-by: Chris Evich <cevich@redhat.com>
@cevich cevich marked this pull request as ready for review September 18, 2023 18:35
@github-actions
Copy link

Cirrus CI build successful. Found built image names and IDs:

Stage Image Name IMAGE_SUFFIX
base fedora do-not-use
cache build-push c20230918t183521z-f38f37d13
cache fedora c20230918t183521z-f38f37d13

@cevich cevich merged commit 09ae91d into containers:main Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no_debian Don't build the debian image no_fedora_podman Don't build the fedora_podman image no_fedora-aws Don't build any fedora-aws images no_fedora-aws-arm64 Don't build any fedora-aws-arm64 images no_fedora-netavark Don't build the fedora-netavark image no_fedora-netavark-aws-arm64 Don't build the fedora-netavark-aws image no_fedora-podman-aws-arm64 Don't build the fedora-podman-aws image no_fedora-podman-py Don't build the fedora-podman-py image no_image-builder Don't build the image-builder image no_prior-fedora_podman Don't build the prior-fedora_podman image no_prior-fedora Don't build any prior-fedora images no_rawhide Don't build the rawhide image no_skopeo_cidev Don't build the skopeo_cidev image no_win-server-wsl Don't build Windows WSL image
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant