Skip to content

Commit d079a72

Browse files
authored
Merge pull request #8 from vistaprint/feature/use-provided-log-group-name
Provide log group name
2 parents 29f0a64 + 74bfd77 commit d079a72

4 files changed

Lines changed: 12 additions & 1 deletion

File tree

cluster.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ resource "aws_eks_cluster" "cluster" {
2525
}
2626

2727
resource "aws_cloudwatch_log_group" "control_plane_logs" {
28-
name = "/aws/eks/${var.cluster_name}/cluster"
28+
name = "/aws/eks/${local.log_group_name}/cluster"
2929
retention_in_days = 30
3030

3131
tags = var.tags

examples/basic/cluster.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ module "cluster" {
44
cluster_name = "simple-eks-integration-test"
55
cluster_version = "1.18"
66
vpc_name = var.vpc_name
7+
log_group_name = "a-test-log-group-name"
78

89
region = var.aws_region
910
profile = var.aws_profile

locals.tf

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
locals {
2+
log_group_name = var.log_group_name == null ? var.cluster_name : var.log_group_name
3+
}

variables.tf

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,13 @@ variable "cluster_name" {
1010
type = string
1111
}
1212

13+
# If this variable is an empty string, the
14+
# log group name will be var.cluster_name
15+
variable "log_group_name" {
16+
type = string
17+
default = null
18+
}
19+
1320
variable "cluster_version" {
1421
type = string
1522
}

0 commit comments

Comments
 (0)