-
Notifications
You must be signed in to change notification settings - Fork 55
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
Fails with docker images pulled by sha #17
Comments
Also, switch to pierrezemb/gostatic:latest instead of specific hash due to satackey/action-docker-layer-caching#17
Also, switch to pierrezemb/gostatic:latest instead of specific hash due to satackey/action-docker-layer-caching#17
I hadn't considered pulling by digest, I'll fix it soon. Thank you for your report! |
I'm getting the same error here:
https://github.com/mdn/kuma/runs/1063849218 My config can be seen here https://github.com/mdn/kuma/pull/7456/files#diff-12a86cef0c4707531fdbabac3e38cb2aR27 if that helps. |
I'm encountering the same issue, any solution for it ? |
That looks like the same issue as the OP; the kuma_base Dockerfile specifies a source image by digest. That image isn't dangling so the action expects it to have a valid tag instead of
Until this issue is resolved I think the only workaround is to specify images by tag instead of by digest. |
One thing to also consider is to keep support for specifying the tag AND the hash. This form is common even though the tag here does absolutely nothing, but only helps the reader:
|
I've just confirmed that docker does treat that form exactly the same as only specifying the hash. That means that it'll just work when the action supports images specified by hash. |
Yes, to Docker:
and
are identical. Just wasn't familiar with this Action's internals and wanted to give a heads up about it :) |
The action currently hash a bug where it won't work at all with images pulled by digest: satackey/action-docker-layer-caching#17
The action currently hash a bug where it won't work at all with images pulled by digest: satackey/action-docker-layer-caching#17
Docker provides a way for referring to images by their SHA hash instead of tags, like this:
Notice the
<none>
part.Now, apparently this causes the docker layer caching action to do the equivalent of following under the hood:
In the logs of Github build I see:
The text was updated successfully, but these errors were encountered: