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

Run pre-built Docker image #195

Merged
1 commit merged into from
Oct 11, 2022
Merged

Run pre-built Docker image #195

1 commit merged into from
Oct 11, 2022

Conversation

ab77
Copy link
Contributor

@ab77 ab77 commented Aug 21, 2022

Given that we are already publishing a Docker image to gh.cr, it makes sense to tell the action to use that, rather than building it from a Dockerfile on every execution. At least this PR makes this possible, if I understood action.yml spec correctly.

Tests:
(build) ~ 56s: https://github.com/belodetek/black.box/runs/7940470251?check_suite_focus=true
(pull) ~ 12s: https://github.com/belodetek/black.box/runs/7940905781?check_suite_focus=true

cc @klutchell @20k-ultra

@ab77 ab77 requested review from 20k-ultra and klutchell August 21, 2022 18:32
action.yml Show resolved Hide resolved
@20k-ultra

This comment was marked as outdated.

@ab77 ab77 closed this Sep 5, 2022
@ab77 ab77 reopened this Sep 5, 2022
@ab77

This comment was marked as outdated.

@klutchell
Copy link
Collaborator

I found the same thing when I tried it @ab77 but I didn't investigate further

@ab77 ab77 force-pushed the ab77/operational branch from 3ebea61 to 4833290 Compare October 10, 2022 15:39
@ab77 ab77 marked this pull request as draft October 10, 2022 15:46
@ab77 ab77 force-pushed the ab77/operational branch 5 times, most recently from 316a89e to d326a4d Compare October 10, 2022 23:11
@ab77 ab77 marked this pull request as ready for review October 10, 2022 23:16
@ab77 ab77 requested review from Page- and klutchell October 10, 2022 23:16
versionist.conf.js Outdated Show resolved Hide resolved
versionist.conf.js Outdated Show resolved Hide resolved
Co-authored-by: Page- <[email protected]>

Change-type: patch
@ab77 ab77 force-pushed the ab77/operational branch from ee862cb to e8769e2 Compare October 11, 2022 14:31
@ab77
Copy link
Contributor Author

ab77 commented Oct 11, 2022

@balena-ci I self-certify!

@ghost ghost merged commit 7ce24e6 into master Oct 11, 2022
@ghost ghost deleted the ab77/operational branch October 11, 2022 16:46
This pull request was closed.
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.

4 participants