Skip to content
This repository has been archived by the owner on Jan 25, 2023. It is now read-only.

Commit

Permalink
Merge pull request #113 from eedwards-sk/GH-112
Browse files Browse the repository at this point in the history
adds optional variable for enabling specified autoscaling group metrics
  • Loading branch information
Etiene authored Nov 27, 2018
2 parents 98e501d + 58f0e2a commit 8417c0f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
2 changes: 2 additions & 0 deletions modules/vault-cluster/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,8 @@ resource "aws_autoscaling_group" "autoscaling_group" {
health_check_grace_period = "${var.health_check_grace_period}"
wait_for_capacity_timeout = "${var.wait_for_capacity_timeout}"

enabled_metrics = ["${var.enabled_metrics}"]

# Use bucket and policies names in tags for depending on them when they are there
# And only create the cluster after S3 bucket and policies exist
# Otherwise Vault might boot and not find the bucket or not yet have the necessary permissions
Expand Down
6 changes: 6 additions & 0 deletions modules/vault-cluster/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -207,3 +207,9 @@ variable "force_destroy_s3_bucket" {
description = "If 'configure_s3_backend' is enabled and you set this to true, when you run terraform destroy, this tells Terraform to delete all the objects in the S3 bucket used for backend storage. You should NOT set this to true in production or you risk losing all your data! This property is only here so automated tests of this module can clean up after themselves. Only used if 'enable_s3_backend' is set to true."
default = false
}

variable "enabled_metrics" {
description = "List of autoscaling group metrics to enable."
type = "list"
default = []
}

0 comments on commit 8417c0f

Please sign in to comment.