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

balena-deploy: Use docker cache when creating compose file #282

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

alexgg
Copy link
Collaborator

@alexgg alexgg commented Jun 13, 2022

If we have just built the hostapp, the container image will be in the
docker cache so try it first.

Also, simplify the arguments to balena_deploy_hostos.

Change-type: patch
Signed-off-by: Alex Gonzalez [email protected]

@alexgg alexgg marked this pull request as draft June 13, 2022 18:31
@alexgg alexgg force-pushed the alexgg/deploy-hostos branch 13 times, most recently from ebd16df to c433c6f Compare June 15, 2022 13:35
It's easier to check its return value if it returns nothing without an
actual match.

Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
@alexgg alexgg force-pushed the alexgg/deploy-hostos branch 9 times, most recently from b0da193 to 238ec5b Compare June 16, 2022 13:51
Use a `--cache-from <block>:<version>` to allow deployment to use images
cached locally, for example when a hostapp or an OS block has been built
and loaded into the local docker cache.

Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
@alexgg alexgg force-pushed the alexgg/deploy-hostos branch from 238ec5b to 0df463a Compare June 16, 2022 15:06
alexgg added 4 commits June 16, 2022 17:19
….inc

The function is only called from within the balena-push-env container
that already contains the balena-docker.inc script include.

Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
Using compose files allows to deploy both single and multi-container apps.

Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>

SQ jenkind_build.sh
Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
@alexgg alexgg force-pushed the alexgg/deploy-hostos branch 2 times, most recently from cccc48f to a7473e3 Compare June 16, 2022 21:09
Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
@alexgg alexgg force-pushed the alexgg/deploy-hostos branch from a7473e3 to 893e75f Compare June 17, 2022 10:07
This allows to use the script outside of the Jenkins context.

Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
@alexgg alexgg force-pushed the alexgg/deploy-hostos branch from 4094af3 to 88a6ce7 Compare June 18, 2022 19:12
alexgg added 2 commits June 21, 2022 15:44
…itories

This allows to use these scripts from container images that contain
the include files in the container root folder.

It simplifies the Dockerfiles for block builder containers.

Change-type: patch
Signed-off-by: Alex Gonzalez <[email protected]>
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.

1 participant