diff --git a/builtin/providers/aws/resource_aws_route53_record.go b/builtin/providers/aws/resource_aws_route53_record.go index 9686c48a90..1966c33de4 100644 --- a/builtin/providers/aws/resource_aws_route53_record.go +++ b/builtin/providers/aws/resource_aws_route53_record.go @@ -407,12 +407,9 @@ func resourceAwsRoute53RecordBuildSet(d *schema.ResourceData, zoneName string) ( rec.HealthCheckId = aws.String(v.(string)) } - if v, ok := d.GetOk("weight"); ok { - rec.Weight = aws.Int64(int64(v.(int))) - } - if v, ok := d.GetOk("set_identifier"); ok { rec.SetIdentifier = aws.String(v.(string)) + rec.Weight = aws.Int64(int64(d.Get("weight").(int))) } return rec, nil diff --git a/builtin/providers/aws/resource_aws_route53_record_test.go b/builtin/providers/aws/resource_aws_route53_record_test.go index 6b25488d37..bbeb859cd8 100644 --- a/builtin/providers/aws/resource_aws_route53_record_test.go +++ b/builtin/providers/aws/resource_aws_route53_record_test.go @@ -133,6 +133,7 @@ func TestAccAWSRoute53Record_weighted(t *testing.T) { Check: resource.ComposeTestCheckFunc( testAccCheckRoute53RecordExists("aws_route53_record.www-dev"), testAccCheckRoute53RecordExists("aws_route53_record.www-live"), + testAccCheckRoute53RecordExists("aws_route53_record.www-off"), ), }, }, @@ -407,6 +408,16 @@ resource "aws_route53_record" "www-live" { set_identifier = "live" records = ["dev.notexample.com"] } + +resource "aws_route53_record" "www-off" { + zone_id = "${aws_route53_zone.main.zone_id}" + name = "www" + type = "CNAME" + ttl = "5" + weight = 0 + set_identifier = "off" + records = ["dev.notexample.com"] +} ` const testAccRoute53ElbAliasRecord = `