-
Notifications
You must be signed in to change notification settings - Fork 595
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 a dangling image prefix when tag name is not specified. #1461
Conversation
fd0e942
to
f1a4107
Compare
@AkihiroSuda can I get a review on this please. |
@manugupt1 Doesn't moby/buildkit#3212 affect this implementation? |
It affects only scratch images or images with no layer. moby/buildkit#3212 (comment) ; unpacking should be skipped for when there are no layers I believe is the right solution in buildkit. Also, this bug comes only when images are built with no tag and has no layers. There is a bug, however, when buildkit fixes that bug; we won't have to fix it on our end with this change later.
|
c24eeb1
to
6f81b76
Compare
@ktock PTAL, this and the PR (moby/buildkit#3251) in buildkit should fix all the concerns so far. LMK if you have questions. |
cmd/nerdctl/build_test.go
Outdated
|
||
func TestBuildNoTag(t *testing.T) { | ||
testutil.RequiresBuild(t) | ||
testutil.DockerIncompatible(t) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why incompatible?
Fixes: containerd#1398 Signed-off-by: Manu Gupta <manugupt1@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
Fixes: #1398