diff --git a/builder/amazon/common/access_config.go b/builder/amazon/common/access_config.go index 4ce478dd9..cb97ff91e 100644 --- a/builder/amazon/common/access_config.go +++ b/builder/amazon/common/access_config.go @@ -149,8 +149,9 @@ func (c *AccessConfig) Prepare(ctx *interpolate.Context) []error { if c.RawRegion != "" && !c.SkipValidation { ec2conn := getValidationSession() - if valid := ValidateRegion(c.RawRegion, ec2conn); !valid { - errs = append(errs, fmt.Errorf("Unknown region: %s", c.RawRegion)) + err := ValidateRegion(c.RawRegion, ec2conn) + if err != nil { + errs = append(errs, fmt.Errorf("error validating region: %s", err.Error())) } } diff --git a/builder/amazon/common/access_config_test.go b/builder/amazon/common/access_config_test.go index 833a6177e..c94d78bff 100644 --- a/builder/amazon/common/access_config_test.go +++ b/builder/amazon/common/access_config_test.go @@ -17,20 +17,20 @@ func TestAccessConfigPrepare_Region(t *testing.T) { mockConn := &mockEC2Client{} c.RawRegion = "us-east-12" - valid := ValidateRegion(c.RawRegion, mockConn) - if valid { + err := ValidateRegion(c.RawRegion, mockConn) + if err == nil { t.Fatalf("should have region validation err: %s", c.RawRegion) } c.RawRegion = "us-east-1" - valid = ValidateRegion(c.RawRegion, mockConn) - if !valid { + err = ValidateRegion(c.RawRegion, mockConn) + if err != nil { t.Fatalf("shouldn't have region validation err: %s", c.RawRegion) } c.RawRegion = "custom" - valid = ValidateRegion(c.RawRegion, mockConn) - if valid { + err = ValidateRegion(c.RawRegion, mockConn) + if err == nil { t.Fatalf("should have region validation err: %s", c.RawRegion) } diff --git a/builder/amazon/common/ami_config.go b/builder/amazon/common/ami_config.go index 58c6b8827..fe3b0949c 100644 --- a/builder/amazon/common/ami_config.go +++ b/builder/amazon/common/ami_config.go @@ -112,8 +112,9 @@ func (c *AMIConfig) prepareRegions(ec2conn ec2iface.EC2API, accessConfig *Access if !c.AMISkipRegionValidation { // Verify the region is real - if valid := ValidateRegion(region, ec2conn); !valid { - errs = append(errs, fmt.Errorf("Unknown region: %s", region)) + err := ValidateRegion(region, ec2conn) + if err != nil { + errs = append(errs, fmt.Errorf("error validating region: %s", err.Error())) } } diff --git a/builder/amazon/common/ami_config_test.go b/builder/amazon/common/ami_config_test.go index 0e4f28e22..9afad469c 100644 --- a/builder/amazon/common/ami_config_test.go +++ b/builder/amazon/common/ami_config_test.go @@ -55,13 +55,17 @@ func TestAMIConfigPrepare_regions(t *testing.T) { c.AMISkipRegionValidation = true var errs []error + var err error mockConn := &mockEC2Client{} if errs = c.prepareRegions(mockConn, nil, errs); len(errs) > 0 { t.Fatalf("shouldn't have err: %#v", errs) } c.AMISkipRegionValidation = false - c.AMIRegions = listEC2Regions(mockConn) + c.AMIRegions, err = listEC2Regions(mockConn) + if err != nil { + t.Fatalf("shouldn't have err: %s", err.Error()) + } if errs = c.prepareRegions(mockConn, nil, errs); len(errs) > 0 { t.Fatalf("shouldn't have err: %#v", errs) } diff --git a/builder/amazon/common/regions.go b/builder/amazon/common/regions.go index 67dee4212..eb8bba19b 100644 --- a/builder/amazon/common/regions.go +++ b/builder/amazon/common/regions.go @@ -1,6 +1,7 @@ package common import ( + "fmt" "github.com/aws/aws-sdk-go/aws/session" "github.com/aws/aws-sdk-go/service/ec2" "github.com/aws/aws-sdk-go/service/ec2/ec2iface" @@ -15,23 +16,32 @@ func getValidationSession() *ec2.EC2 { return ec2conn } -func listEC2Regions(ec2conn ec2iface.EC2API) []string { +func listEC2Regions(ec2conn ec2iface.EC2API) ([]string, error) { var regions []string - resultRegions, _ := ec2conn.DescribeRegions(nil) + resultRegions, err := ec2conn.DescribeRegions(nil) + if err != nil { + return []string{}, err + } for _, region := range resultRegions.Regions { regions = append(regions, *region.RegionName) } - return regions + return regions, nil } // ValidateRegion returns true if the supplied region is a valid AWS // region and false if it's not. -func ValidateRegion(region string, ec2conn ec2iface.EC2API) bool { - for _, valid := range listEC2Regions(ec2conn) { +func ValidateRegion(region string, ec2conn ec2iface.EC2API) error { + regions, err := listEC2Regions(ec2conn) + if err != nil { + return err + } + + for _, valid := range regions { if region == valid { - return true + return nil } } - return false + + return fmt.Errorf("Invalid region: %s", region) }