Skip to content

fix: add --load flag for local buildkit#9387

Merged
ericzzzzzzz merged 2 commits intoGoogleContainerTools:mainfrom
Crystalix007:main
Apr 17, 2024
Merged

fix: add --load flag for local buildkit#9387
ericzzzzzzz merged 2 commits intoGoogleContainerTools:mainfrom
Crystalix007:main

Conversation

@Crystalix007
Copy link
Copy Markdown
Contributor

Description

Adds the --load flag when building images using buildkit, and not pushing to a remote registry.

This is required, to get them loaded into the local docker context.

User facing changes (remove if N/A)

Before:

Running dev complained with:

WARNING: No output specified with docker-container driver. Build result will only remain in the build cache. To push result image into registry use --push or to load image into docker use --load
...
Build [docker.io/...] failed: could not push image "docker.io/...:4002597-dirty": An image does not exist locally with the tag: ...

After:

Tags used in deployment:
 - docker.io/... -> docker.io/crystalix007/...:e1958765598cf4713ec0f5c9d9afa702fccc6098d509d84d272a14b373f422dd

Adds the --load flag when building images using buildkit, and not
pushing to a remote registry.

This is required, to get them loaded into the local docker context.
Adds testing to ensure that the --load flag is applied correctly.
@google-cla
Copy link
Copy Markdown

google-cla bot commented Apr 13, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@ericzzzzzzz ericzzzzzzz added the kokoro:force-run forces a kokoro re-run on a PR label Apr 15, 2024
@kokoro-team kokoro-team removed the kokoro:force-run forces a kokoro re-run on a PR label Apr 15, 2024
@ericzzzzzzz
Copy link
Copy Markdown
Contributor

Hi @Crystalix007 Thank you for the contribution! The change looks good, but skaffold integration test infra seems broken at the moment, we'll merge the change after the issue is fixed.

@ericzzzzzzz ericzzzzzzz added the kokoro:force-run forces a kokoro re-run on a PR label Apr 16, 2024
@kokoro-team kokoro-team removed the kokoro:force-run forces a kokoro re-run on a PR label Apr 16, 2024
@ericzzzzzzz ericzzzzzzz added the kokoro:force-run forces a kokoro re-run on a PR label Apr 17, 2024
@kokoro-team kokoro-team removed the kokoro:force-run forces a kokoro re-run on a PR label Apr 17, 2024
@ericzzzzzzz ericzzzzzzz merged commit a2f4043 into GoogleContainerTools:main Apr 17, 2024
@menahyouyeah menahyouyeah mentioned this pull request Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants