Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Commit 6267dc4

Browse files
committed
Handle images without an explicit tag
Signed-off-by: Nicolas De Loof <[email protected]>
1 parent 163c9e0 commit 6267dc4

File tree

6 files changed

+8
-8
lines changed

6 files changed

+8
-8
lines changed

e2e/images_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,11 @@ import (
1414

1515
func insertBundles(t *testing.T, cmd icmd.Cmd, info dindSwarmAndRegistryInfo) {
1616
// Push an application so that we can later pull it by digest
17-
cmd.Command = dockerCli.Command("app", "build", "--tag", info.registryAddress+"/c-myapp", filepath.Join("testdata", "push-pull"))
17+
cmd.Command = dockerCli.Command("app", "build", "--no-resolve-image", "--tag", info.registryAddress+"/c-myapp", filepath.Join("testdata", "push-pull"))
1818
icmd.RunCmd(cmd).Assert(t, icmd.Success)
19-
cmd.Command = dockerCli.Command("app", "build", "--tag", "b-simple-app", filepath.Join("testdata", "simple"))
19+
cmd.Command = dockerCli.Command("app", "build", "--no-resolve-image", "--tag", "b-simple-app", filepath.Join("testdata", "simple"))
2020
icmd.RunCmd(cmd).Assert(t, icmd.Success)
21-
cmd.Command = dockerCli.Command("app", "build", "--tag", "a-simple-app", filepath.Join("testdata", "simple"))
21+
cmd.Command = dockerCli.Command("app", "build", "--no-resolve-image", "--tag", "a-simple-app", filepath.Join("testdata", "simple"))
2222
icmd.RunCmd(cmd).Assert(t, icmd.Success)
2323
}
2424

e2e/testdata/app-inspect.golden

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
},
2828
{
2929
"Name": "web",
30-
"Image": "nginx:latest",
30+
"Image": "nginx",
3131
"Replicas": 1,
3232
"Ports": "8082"
3333
}

e2e/testdata/expected-json-render.golden

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@
4747
"resources": {},
4848
"placement": {}
4949
},
50-
"image": "nginx:latest",
50+
"image": "nginx",
5151
"networks": {
5252
"front": null
5353
},

e2e/testdata/expected-yaml-render.golden

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ services:
1313
networks:
1414
back: null
1515
web:
16-
image: nginx:latest
16+
image: nginx
1717
networks:
1818
front: null
1919
ports:

e2e/testdata/simple/simple.dockerapp/docker-compose.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ services:
99
- api.example.com
1010
- ${api_host}
1111
web:
12-
image: nginx:latest
12+
image: nginx
1313
networks:
1414
- front
1515
volumes:

internal/commands/build/build.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ func fixServiceImageReferences(ctx context.Context, dockerCli command.Cli, bundl
200200
resolver := remotes.CreateResolver(dockerCli.ConfigFile(), insecureRegistries...)
201201
for _, service := range pulledServices {
202202
image := bundle.Images[service.Name]
203-
ref, err := reference.ParseNormalizedNamed(*service.Image)
203+
ref, err := reference.ParseDockerRef(*service.Image)
204204
if err != nil {
205205
return errors.Wrapf(err, "could not resolve image %s", *service.Image)
206206
}

0 commit comments

Comments
 (0)