providers/aws: Move HostedZoneIDForRegion into TF

builtin/providers/aws/hosted_zones.go

@@ -0,0 +1,23 @@
+package aws
+
+// http://docs.aws.amazon.com/general/latest/gr/rande.html#s3_website_region_endpoints
+var hostedZoneIDsMap = map[string]string{
+	"us-east-1":      "Z3AQBSTGFYJSTF",
+	"us-west-2":      "Z3BJ6K6RIION7M",
+	"us-west-1":      "Z2F56UZL2M1ACD",
+	"eu-west-1":      "Z1BKCTXD74EZPE",
+	"central-1":      "Z21DNDUVLTQW6Q",
+	"ap-southeast-1": "Z3O0J2DXBE1FTB",
+	"ap-southeast-2": "Z1WCIGYICN2BYD",
+	"ap-northeast-1": "Z2M4EHUR26P7ZW",
+	"sa-east-1":      "Z7KQH4QJS55SO",
+	"us-gov-west-1":  "Z31GFT0UA1I2HV",
+}
+
+func HostedZoneIDForRegion(region string) string {
+	if region == "" {
+		region = "us-east-1"
+	}
+
+	return hostedZoneIDsMap[region]
+}

builtin/providers/aws/hosted_zones_test.go

@@ -0,0 +1,24 @@
+package aws
+
+import (
+	"testing"
+)
+
+func TestHostedZoneIDForRegion(t *testing.T) {
+	if r := HostedZoneIDForRegion("us-east-1"); r != "Z3AQBSTGFYJSTF" {
+		t.Fatalf("bad: %s", r)
+	}
+	if r := HostedZoneIDForRegion("ap-southeast-2"); r != "Z1WCIGYICN2BYD" {
+		t.Fatalf("bad: %s", r)
+	}
+
+	// Empty string should default to us-east-1
+	if r := HostedZoneIDForRegion(""); r != "Z3AQBSTGFYJSTF" {
+		t.Fatalf("bad: %s", r)
+	}
+
+	// Bad input should be empty string
+	if r := HostedZoneIDForRegion("not-a-region"); r != "" {
+		t.Fatalf("bad: %s", r)
+	}
+}

builtin/providers/aws/resource_aws_s3_bucket.go

@@ -176,7 +176,7 @@ func resourceAwsS3BucketRead(d *schema.ResourceData, meta interface{}) error {
 	}
 
 	// Add the hosted zone ID for this bucket's region as an attribute
-	hostedZoneID := s3.HostedZoneIDForRegion(region)
+	hostedZoneID := HostedZoneIDForRegion(region)
 	if err := d.Set("hosted_zone_id", hostedZoneID); err != nil {
 		return err
 	}