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

Add support for remote registry cache #94

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

v-yarotsky
Copy link

@v-yarotsky v-yarotsky commented Jun 22, 2022

This pull requests has a few things:

  • Adds support for registry-based caching (continuing what has been attempted in Adding an optional REGISTRY_CACHE to allow for caching between different hosts #17)
  • Adds support for insecure registries (can be useful if one wants to use a locally deployed registry for caching).
  • Fixes the contents of the ./digest file – previously it was populated with config manifest digest, not the image digest, meaning that it couldn't be used to fetch the image!

Tests pass locally:

PASS
ok      github.com/concourse/oci-build-task     15.618s

@v-yarotsky v-yarotsky force-pushed the vy-registry-image-cache branch 2 times, most recently from 61c66d5 to 4c6815f Compare June 22, 2022 23:09
Vladimir Yarotsky added 5 commits June 24, 2022 15:34
Signed-off-by: Vladimir Yarotsky <[email protected]>
Signed-off-by: Vladimir Yarotsky <[email protected]>
Signed-off-by: Vladimir Yarotsky <[email protected]>
Signed-off-by: Vladimir Yarotsky <[email protected]>
@v-yarotsky v-yarotsky force-pushed the vy-registry-image-cache branch from e0c694d to db22b1c Compare June 24, 2022 22:34
@v-yarotsky v-yarotsky marked this pull request as ready for review June 24, 2022 22:40
@kvidzibo
Copy link

kvidzibo commented Nov 8, 2022

Hello, I was wondering is there any update on this? are there any plans of merging it or will it get 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.

3 participants