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

Fix: propagate --cache-from to buildx build #638

Merged
merged 2 commits into from
Apr 12, 2024

Conversation

dhhyi
Copy link
Contributor

@dhhyi dhhyi commented Sep 13, 2023

I am trying to use the new --cache-to feature from #570 with a registry cache.

If I understand it correctly this only works when using both --cache-to and --cache-from, but the devcontainers-cli does not send --cache-from to docker buildx build.

@cjam
Copy link

cjam commented Mar 13, 2024

Does this resolve the issue of devcontainer features being rebuilt everytime the devcontainer is used irrespective of prebuilt container?

Copy link
Contributor

@chrmarti chrmarti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Left a few comments.

src/spec-node/singleContainer.ts Outdated Show resolved Hide resolved
src/spec-node/containerFeatures.ts Outdated Show resolved Hide resolved
@dhhyi dhhyi requested a review from chrmarti March 20, 2024 15:10
Copy link
Contributor

@chrmarti chrmarti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Member

@samruddhikhandale samruddhikhandale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@samruddhikhandale samruddhikhandale merged commit 9d98f30 into devcontainers:main Apr 12, 2024
19 checks passed
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