Skip to content

Commit

Permalink
Merge pull request #35 from rackspace-infrastructure-automation/concu…
Browse files Browse the repository at this point in the history
…rrent-customer-alarms

Permit customer alarms to be explicitly enabled instead of implicitly…
  • Loading branch information
jp-rivas authored Jun 28, 2021
2 parents 4371e4e + 2aa8962 commit 68f3529
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 0 deletions.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,8 @@ The following module variables were updated to better meet current Rackspace sty
| add\_waf | Add an existing Regional WAF to the ALB. true \| false | `bool` | `false` | no |
| create\_internal\_zone\_record | Create Route 53 internal zone record for the ALB. i.e true \| false | `bool` | `false` | no |
| create\_logging\_bucket | Create a new S3 logging bucket. i.e. true \| false | `bool` | `true` | no |
| customer\_alarms\_cleared | Specifies whether alarms will notify customers when returning to an OK status. | `bool` | `false` | no |
| customer\_alarms\_enabled | Specifies whether alarms will notify customers. Automatically enabled if rackspace\_managed is set to false | `bool` | `false` | no |
| enable\_deletion\_protection | If true, deletion of the load balancer will be disabled via the AWS API. This will prevent Terraform from deleting the load balancer. Defaults to false. | `bool` | `false` | no |
| enable\_http2 | If true sets HTTP/2 to enabled. | `bool` | `true` | no |
| enable\_https\_redirect | If true and at least one HTTP and one HTTPS listener is created, HTTP listeners will have a redirect rule created to forward all traffic to the first HTTPS listener. | `bool` | `false` | no |
Expand Down
2 changes: 2 additions & 0 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -330,6 +330,8 @@ module "unhealthy_host_count_alarm" {
alarm_description = "Unhealthy Host count is greater than or equal to threshold, creating ticket."
alarm_name = "${var.name}_unhealthy_host_count_alarm"
comparison_operator = "GreaterThanOrEqualToThreshold"
customer_alarms_cleared = var.customer_alarms_cleared
customer_alarms_enabled = var.customer_alarms_enabled
dimensions = data.null_data_source.alarm_dimensions.*.outputs
evaluation_periods = 10
metric_name = "UnHealthyHostCount"
Expand Down
12 changes: 12 additions & 0 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,18 @@ variable "create_logging_bucket" {
default = true
}

variable "customer_alarms_cleared" {
description = "Specifies whether alarms will notify customers when returning to an OK status."
type = bool
default = false
}

variable "customer_alarms_enabled" {
description = "Specifies whether alarms will notify customers. Automatically enabled if rackspace_managed is set to false"
type = bool
default = false
}

variable "enable_deletion_protection" {
description = "If true, deletion of the load balancer will be disabled via the AWS API. This will prevent Terraform from deleting the load balancer. Defaults to false."
type = bool
Expand Down

0 comments on commit 68f3529

Please sign in to comment.