-
Notifications
You must be signed in to change notification settings - Fork 17
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
Bump automation library version #348
Conversation
LGTM. I assume this addresses the issue you reported in your |
Thanks, and yes. |
8099349
to
28b7868
Compare
Ugh, that |
Importantly, this contains a necessary fix for `build_push.sh` needed to stop immutable-image existence-check failing on build (c/image_build cron job). Signed-off-by: Chris Evich <cevich@redhat.com>
Previously this task depended on executing a downstream test script intended for exercising an orthagonal orchestration script (which happens to call `build-push.sh`. Having upstream CI VM image builds depend on a downstream script is very much not ideal. Replace this with a very quick/dirty test that simply confirms a multi-arch build can function. Signed-off-by: Chris Evich <cevich@redhat.com>
Many/most of the container image builds rely on pulling packages from repos that are sometimes slow/busy. Give the tasks a bit of extra time in case it's needed. Signed-off-by: Chris Evich <cevich@redhat.com>
28b7868
to
c3a0ca1
Compare
Cirrus CI build successful. Found built image names and IDs:
|
Oops, I missed the
|
Thanks Ed, no worries. I just needed the build-push VM image to fix our ailing 'immutable' builds. The real interesting stuff is going to be in #349 |
Importantly, this contains a necessary fix for
build_push.sh
needed to stop immutable-image existence-check failing on build (c/image_build cron job).