Commit 1ab6569d authored by ampsingram's avatar ampsingram

format changes

parent 28d37740
...@@ -615,14 +615,14 @@ func (cfg *CloudConfig) validateOverrides() error { ...@@ -615,14 +615,14 @@ func (cfg *CloudConfig) validateOverrides() error {
ovrd.Region = region ovrd.Region = region
url := strings.TrimSpace(ovrd.URL) url := strings.TrimSpace(ovrd.URL)
if url== "" { if url == "" {
return fmt.Errorf("url is missing [URL is \"\"] in override %s", onum) return fmt.Errorf("url is missing [URL is \"\"] in override %s", onum)
} }
signingRegion := strings.TrimSpace(ovrd.SigningRegion) signingRegion := strings.TrimSpace(ovrd.SigningRegion)
if signingRegion == "" { if signingRegion == "" {
return fmt.Errorf("signingRegion is missing [SigningRegion is \"\"] in override %s", onum) return fmt.Errorf("signingRegion is missing [SigningRegion is \"\"] in override %s", onum)
} }
signature := name+"_"+region signature := name + "_" + region
if set[signature] { if set[signature] {
return fmt.Errorf("duplicate entry found for service override [%s] (%s in %s)", onum, name, region) return fmt.Errorf("duplicate entry found for service override [%s] (%s in %s)", onum, name, region)
} }
......
...@@ -309,7 +309,7 @@ func TestOverridesActiveConfig(t *testing.T) { ...@@ -309,7 +309,7 @@ func TestOverridesActiveConfig(t *testing.T) {
SigningMethod = v4`), SigningMethod = v4`),
nil, nil,
false, true, false, true,
[]ServiceDescriptor{{name:"s3", region: "sregion1", signingRegion: "sregion1", signingMethod: "v4"}, []ServiceDescriptor{{name: "s3", region: "sregion1", signingRegion: "sregion1", signingMethod: "v4"},
{name: "ec2", region: "sregion2", signingRegion: "sregion2", signingMethod: "v4"}}, {name: "ec2", region: "sregion2", signingRegion: "sregion2", signingMethod: "v4"}},
}, },
{ {
...@@ -356,7 +356,7 @@ func TestOverridesActiveConfig(t *testing.T) { ...@@ -356,7 +356,7 @@ func TestOverridesActiveConfig(t *testing.T) {
nil, nil,
false, true, false, true,
[]ServiceDescriptor{{name: "s3", region: "region1", signingRegion: "sregion1", signingMethod: ""}, []ServiceDescriptor{{name: "s3", region: "region1", signingRegion: "sregion1", signingMethod: ""},
{name:"ec2", region: "region2", signingRegion: "sregion", signingMethod: "v4"}}, {name: "ec2", region: "region2", signingRegion: "sregion", signingMethod: "v4"}},
}, },
{ {
"Multiple regions, Same Service", "Multiple regions, Same Service",
...@@ -380,8 +380,8 @@ func TestOverridesActiveConfig(t *testing.T) { ...@@ -380,8 +380,8 @@ func TestOverridesActiveConfig(t *testing.T) {
`), `),
nil, nil,
false, true, false, true,
[]ServiceDescriptor{{name:"s3", region: "region1", signingRegion: "sregion1", signingMethod: "v3"}, []ServiceDescriptor{{name: "s3", region: "region1", signingRegion: "sregion1", signingMethod: "v3"},
{name:"s3", region: "region2", signingRegion: "sregion1", signingMethod: "v4"}}, {name: "s3", region: "region2", signingRegion: "sregion1", signingMethod: "v4"}},
}, },
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment