Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

Commit 4a4e6be

Browse files
authored
Merge pull request #1124 from docker/aci_resolve_containername_alias
Support container_name in ACI and add network alias in dns sidecar.
2 parents d2d3a94 + 010c4be commit 4a4e6be

File tree

3 files changed

+18
-8
lines changed

3 files changed

+18
-8
lines changed

aci/convert/convert.go

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,8 @@ func ToContainerGroup(ctx context.Context, aciContext store.AciContext, p types.
120120
DNSNameLabel: dnsLabelName,
121121
}
122122
}
123-
if len(ctnrs) > 1 {
124-
dnsSideCar := getDNSSidecar(ctnrs)
123+
if len(project.Services) > 1 {
124+
dnsSideCar := getDNSSidecar(project.Services)
125125
ctnrs = append(ctnrs, dnsSideCar)
126126
}
127127
groupDefinition.ContainerGroupProperties.Containers = &ctnrs
@@ -137,10 +137,13 @@ func durationToSeconds(d *types.Duration) *int32 {
137137
return &v
138138
}
139139

140-
func getDNSSidecar(containers []containerinstance.Container) containerinstance.Container {
140+
func getDNSSidecar(services types.Services) containerinstance.Container {
141141
names := []string{"/hosts"}
142-
for _, container := range containers {
143-
names = append(names, *container.Name)
142+
for _, service := range services {
143+
names = append(names, service.Name)
144+
if service.ContainerName != "" {
145+
names = append(names, service.ContainerName)
146+
}
144147
}
145148
dnsSideCar := containerinstance.Container{
146149
Name: to.StringPtr(ComposeDNSSidecarName),
@@ -182,8 +185,13 @@ func (s serviceConfigAciHelper) getAciContainer() (containerinstance.Container,
182185
return containerinstance.Container{}, err
183186
}
184187

188+
containerName := s.Name
189+
if s.ContainerName != "" {
190+
containerName = s.ContainerName
191+
}
192+
185193
return containerinstance.Container{
186-
Name: to.StringPtr(s.Name),
194+
Name: to.StringPtr(containerName),
187195
ContainerProperties: &containerinstance.ContainerProperties{
188196
Image: to.StringPtr(s.Image),
189197
Command: to.StringSlicePtr(s.Command),

tests/compose-e2e/fixtures/volume-test/docker-compose.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,5 @@ services:
1515
- 9090:80
1616

1717
volumes:
18-
staticVol:
18+
staticVol:
19+
name: myVolume

tests/composefiles/demo_multi_port.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@ services:
33
build: aci-demo/db
44
image: gtardif/sentences-db
55

6-
words:
6+
service1:
7+
container_name: words
78
build: aci-demo/words
89
image: gtardif/sentences-api
910
ports:

0 commit comments

Comments
 (0)