From 3ebbb62bb0f0d0638eb7805d70c8ffbdad878a02 Mon Sep 17 00:00:00 2001 From: stack72 Date: Tue, 3 Nov 2015 23:28:47 +0000 Subject: [PATCH] Adding child_healthchecks to the Route 53 HealthCheck resource --- .../aws/resource_aws_route53_health_check.go | 46 +++++++++++++++++-- .../resource_aws_route53_health_check_test.go | 37 +++++++++++++++ .../aws/r/route53_health_check.html.markdown | 14 +++++- 3 files changed, 91 insertions(+), 6 deletions(-) diff --git a/builtin/providers/aws/resource_aws_route53_health_check.go b/builtin/providers/aws/resource_aws_route53_health_check.go index 95859dc0b..b9db3050f 100644 --- a/builtin/providers/aws/resource_aws_route53_health_check.go +++ b/builtin/providers/aws/resource_aws_route53_health_check.go @@ -1,6 +1,7 @@ package aws import ( + "fmt" "log" "time" @@ -26,11 +27,11 @@ func resourceAwsRoute53HealthCheck() *schema.Resource { }, "failure_threshold": &schema.Schema{ Type: schema.TypeInt, - Required: true, + Optional: true, }, "request_interval": &schema.Schema{ Type: schema.TypeInt, - Required: true, + Optional: true, ForceNew: true, // todo this should be updateable but the awslabs route53 service doesnt have the ability }, "ip_address": &schema.Schema{ @@ -68,6 +69,25 @@ func resourceAwsRoute53HealthCheck() *schema.Resource { ForceNew: true, }, + "child_healthchecks": &schema.Schema{ + Type: schema.TypeSet, + Elem: &schema.Schema{Type: schema.TypeString}, + Optional: true, + Set: schema.HashString, + }, + "child_health_threshold": &schema.Schema{ + Type: schema.TypeInt, + Optional: true, + ValidateFunc: func(v interface{}, k string) (ws []string, es []error) { + value := v.(int) + if value > 256 { + es = append(es, fmt.Errorf( + "Child HealthThreshold cannot be more than 256")) + } + return + }, + }, + "tags": tagsSchema(), }, } @@ -100,6 +120,14 @@ func resourceAwsRoute53HealthCheckUpdate(d *schema.ResourceData, meta interface{ updateHealthCheck.Inverted = aws.Bool(d.Get("invert_healthcheck").(bool)) } + if d.HasChange("child_healthchecks") { + updateHealthCheck.ChildHealthChecks = expandStringList(d.Get("child_healthchecks").(*schema.Set).List()) + + } + if d.HasChange("child_health_threshold") { + updateHealthCheck.HealthThreshold = aws.Int64(int64(d.Get("child_health_threshold").(int))) + } + _, err := conn.UpdateHealthCheck(updateHealthCheck) if err != nil { return err @@ -116,9 +144,15 @@ func resourceAwsRoute53HealthCheckCreate(d *schema.ResourceData, meta interface{ conn := meta.(*AWSClient).r53conn healthConfig := &route53.HealthCheckConfig{ - Type: aws.String(d.Get("type").(string)), - FailureThreshold: aws.Int64(int64(d.Get("failure_threshold").(int))), - RequestInterval: aws.Int64(int64(d.Get("request_interval").(int))), + Type: aws.String(d.Get("type").(string)), + } + + if v, ok := d.GetOk("request_interval"); ok { + healthConfig.RequestInterval = aws.Int64(int64(v.(int))) + } + + if v, ok := d.GetOk("failure_threshold"); ok { + healthConfig.FailureThreshold = aws.Int64(int64(v.(int))) } if v, ok := d.GetOk("fqdn"); ok { @@ -209,6 +243,8 @@ func resourceAwsRoute53HealthCheckRead(d *schema.ResourceData, meta interface{}) d.Set("resource_path", updated.ResourcePath) d.Set("measure_latency", updated.MeasureLatency) d.Set("invent_healthcheck", updated.Inverted) + d.Set("child_healthchecks", updated.ChildHealthChecks) + d.Set("child_health_threshold", updated.HealthThreshold) // read the tags req := &route53.ListTagsForResourceInput{ diff --git a/builtin/providers/aws/resource_aws_route53_health_check_test.go b/builtin/providers/aws/resource_aws_route53_health_check_test.go index d199f4e9b..c3f81de6a 100644 --- a/builtin/providers/aws/resource_aws_route53_health_check_test.go +++ b/builtin/providers/aws/resource_aws_route53_health_check_test.go @@ -40,6 +40,22 @@ func TestAccAWSRoute53HealthCheck_basic(t *testing.T) { }) } +func TestAccAWSRoute53HealthCheck_withChildHealthChecks(t *testing.T) { + resource.Test(t, resource.TestCase{ + PreCheck: func() { testAccPreCheck(t) }, + Providers: testAccProviders, + CheckDestroy: testAccCheckRoute53HealthCheckDestroy, + Steps: []resource.TestStep{ + resource.TestStep{ + Config: testAccRoute53HealthCheckConfig_withChildHealthChecks, + Check: resource.ComposeTestCheckFunc( + testAccCheckRoute53HealthCheckExists("aws_route53_health_check.foo"), + ), + }, + }, + }) +} + func TestAccAWSRoute53HealthCheck_IpConfig(t *testing.T) { resource.Test(t, resource.TestCase{ PreCheck: func() { testAccPreCheck(t) }, @@ -170,3 +186,24 @@ resource "aws_route53_health_check" "bar" { } } ` + +const testAccRoute53HealthCheckConfig_withChildHealthChecks = ` +resource "aws_route53_health_check" "child1" { + fqdn = "child1.notexample.com" + port = 80 + type = "HTTP" + resource_path = "/" + failure_threshold = "2" + request_interval = "30" +} + +resource "aws_route53_health_check" "foo" { + type = "CALCULATED" + child_health_threshold = 1 + child_healthchecks = ["${aws_route53_health_check.child1.id}"] + + tags = { + Name = "tf-test-calculated-health-check" + } +} +` diff --git a/website/source/docs/providers/aws/r/route53_health_check.html.markdown b/website/source/docs/providers/aws/r/route53_health_check.html.markdown index dd16e15e1..ad2221726 100644 --- a/website/source/docs/providers/aws/r/route53_health_check.html.markdown +++ b/website/source/docs/providers/aws/r/route53_health_check.html.markdown @@ -12,7 +12,7 @@ Provides a Route53 health check. ## Example Usage ``` -resource "aws_route53_health_check" "foo" { +resource "aws_route53_health_check" "child1" { fqdn = "foobar.terraform.com" port = 80 type = "HTTP" @@ -24,6 +24,16 @@ resource "aws_route53_health_check" "foo" { Name = "tf-test-health-check" } } + +resource "aws_route53_health_check" "foo" { + type = "CALCULATED" + child_health_threshold = 1 + child_healthchecks = ["${aws_route53_health_check.child1.id}"] + + tags = { + Name = "tf-test-calculated-health-check" + } +} ``` ## Argument Reference @@ -38,6 +48,8 @@ The following arguments are supported: * `search_string` - (Optional) String searched in respoonse body for check to considered healthy. * `measure_latency` - (Optional) A Boolean value that indicates whether you want Route 53 to measure the latency between health checkers in multiple AWS regions and your endpoint and to display CloudWatch latency graphs in the Route 53 console. * `invert_healthcheck` - (Optional) A boolean value that indicates whether the status of health check should be inverted. For example, if a health check is healthy but Inverted is True , then Route 53 considers the health check to be unhealthy. +* `child_healthchecks` - (Optional) For a specified parent health check, a list of HealthCheckId values for the associated child health checks. +* `child_health_threshold` - (Optional) The minimum number of child health checks that must be healthy for Route 53 to consider the parent health check to be healthy. Valid values are integers between 0 and 256, inclusive * `tags` - (Optional) A mapping of tags to assign to the health check. At least one of either `fqdn` or `ip_address` must be specified.