Merge pull request #623 from docker/external

Prefer external.name over x-aws-securitygroup
This commit is contained in:
Nicolas De loof 2020-09-17 10:22:28 +02:00 committed by GitHub
commit 387370d074
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 0 deletions

View File

@ -566,7 +566,11 @@ func createCloudMap(project *types.Project, template *cloudformation.Template) {
}
func convertNetwork(project *types.Project, net types.NetworkConfig, vpc string, template *cloudformation.Template) string {
if net.External.External {
return net.Name
}
if sg, ok := net.Extensions[extensionSecurityGroup]; ok {
logrus.Warn("to use an existing security-group, set `network.external` and `network.name` in your compose file")
logrus.Debugf("Security Group for network %q set by user to %q", net.Name, sg)
return sg.(string)
}