Skip to content

Commit af4278f

Browse files
authored
Merge pull request #851 from containerd/dependabot/go_modules/github.com/compose-spec/compose-go-1.1.0
Bump github.com/compose-spec/compose-go from 1.0.9 to 1.1.0
2 parents f03838c + e8f7b6a commit af4278f

File tree

5 files changed

+12
-9
lines changed

5 files changed

+12
-9
lines changed

go.mod

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ go 1.17
44

55
require (
66
github.com/Microsoft/go-winio v0.5.2
7-
github.com/compose-spec/compose-go v1.0.9
7+
github.com/compose-spec/compose-go v1.1.0
88
github.com/containerd/cgroups v1.0.3
99
github.com/containerd/console v1.0.3
1010
github.com/containerd/containerd v1.6.1

go.sum

+2-2
Original file line numberDiff line numberDiff line change
@@ -252,8 +252,8 @@ github.com/cockroachdb/datadriven v0.0.0-20190809214429-80d97fb3cbaa/go.mod h1:z
252252
github.com/cockroachdb/datadriven v0.0.0-20200714090401-bf6692d28da5/go.mod h1:h6jFvWxBdQXxjopDMZyH2UVceIRfR84bdzbkoKrsWNo=
253253
github.com/cockroachdb/errors v1.2.4/go.mod h1:rQD95gz6FARkaKkQXUksEje/d9a6wBJoCr5oaCLELYA=
254254
github.com/cockroachdb/logtags v0.0.0-20190617123548-eb05cc24525f/go.mod h1:i/u985jwjWRlyHXQbwatDASoW0RMlZ/3i9yJHE2xLkI=
255-
github.com/compose-spec/compose-go v1.0.9 h1:mLEdaXkxgO7rUJEa+WoFjwl08LuoW7PeYwey74Z3jao=
256-
github.com/compose-spec/compose-go v1.0.9/go.mod h1:rHEHEY6CRqTAsRy+qvxWSr5GmqzgLWspRD0zloT9GOk=
255+
github.com/compose-spec/compose-go v1.1.0 h1:n/0O7Fl90doi5AG9bfF0SOSLXtveXxLkGXblSrxVC5Q=
256+
github.com/compose-spec/compose-go v1.1.0/go.mod h1:pAy7Mikpeft4pxkFU565/DRHEbDfR84G6AQuiL+Hdg8=
257257
github.com/containerd/aufs v0.0.0-20200908144142-dab0cbea06f4/go.mod h1:nukgQABAEopAHvB6j7cnP5zJ+/3aVcE7hCYqvIwAHyE=
258258
github.com/containerd/aufs v0.0.0-20201003224125-76a6863f2989/go.mod h1:AkGGQs9NM2vtYHaUen+NljV0/baGCAPELGm2q9ZXpWU=
259259
github.com/containerd/aufs v0.0.0-20210316121734-20793ff83c97/go.mod h1:kL5kd6KM5TzQjR79jljyi4olc1Vrx6XBlcyj3gNv2PU=

pkg/composer/projectloader/projectloader.go

+4-1
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,9 @@ import (
2525
)
2626

2727
func Load(fileName, projectName string, envMap map[string]string) (*compose.Project, error) {
28+
if envMap == nil {
29+
envMap = make(map[string]string)
30+
}
2831
b, err := os.ReadFile(fileName)
2932
if err != nil {
3033
return nil, err
@@ -45,6 +48,6 @@ func Load(fileName, projectName string, envMap map[string]string) (*compose.Proj
4548

4649
func withProjectName(name string) func(*loader.Options) {
4750
return func(lOpts *loader.Options) {
48-
lOpts.Name = name
51+
lOpts.SetProjectName(name, true)
4952
}
5053
}

pkg/composer/serviceparser/serviceparser.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -632,13 +632,13 @@ func servicePortConfigToFlagP(c types.ServicePortConfig) (string, error) {
632632
default:
633633
return "", fmt.Errorf("unsupported port mode: %s", c.Mode)
634634
}
635-
if c.Published <= 0 {
636-
return "", fmt.Errorf("unsupported port number: %d", c.Published)
635+
if c.Published == "" {
636+
return "", fmt.Errorf("unsupported port number: %q", c.Published)
637637
}
638638
if c.Target <= 0 {
639639
return "", fmt.Errorf("unsupported port number: %d", c.Target)
640640
}
641-
s := fmt.Sprintf("%d:%d", c.Published, c.Target)
641+
s := fmt.Sprintf("%s:%d", c.Published, c.Target)
642642
if c.HostIP != "" {
643643
if strings.Contains(c.HostIP, ":") {
644644
s = fmt.Sprintf("[%s]:%s", c.HostIP, s)

pkg/composer/serviceparser/serviceparser_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ func TestServicePortConfigToFlagP(t *testing.T) {
4040
ServicePortConfig: types.ServicePortConfig{
4141
Mode: "ingress",
4242
Target: 80,
43-
Published: 8080,
43+
Published: "8080",
4444
Protocol: "tcp",
4545
},
4646
expected: "8080:80/tcp",
@@ -49,7 +49,7 @@ func TestServicePortConfigToFlagP(t *testing.T) {
4949
ServicePortConfig: types.ServicePortConfig{
5050
HostIP: "127.0.0.1",
5151
Target: 80,
52-
Published: 8080,
52+
Published: "8080",
5353
},
5454
expected: "127.0.0.1:8080:80",
5555
},

0 commit comments

Comments
 (0)