Skip to content

Commit

Permalink
fix: set log retention in each module by default to 180 (philips-labs…
Browse files Browse the repository at this point in the history
…#3433)

Top level module has set log retention to 180 days. This PR's ensure
that each module is applying the same default.
  • Loading branch information
npalm authored Aug 16, 2023
1 parent 446dfec commit 86e6abc
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion modules/multi-runner/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ variable "role_path" {
variable "logging_retention_in_days" {
description = "Specifies the number of days you want to retain log events for the lambda log group. Possible values are: 0, 1, 3, 5, 7, 14, 30, 60, 90, 120, 150, 180, 365, 400, 545, 731, 1827, and 3653."
type = number
default = 7
default = 180
}

variable "logging_kms_key_id" {
Expand Down
2 changes: 1 addition & 1 deletion modules/runner-binaries-syncer/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ variable "runner_architecture" {
variable "logging_retention_in_days" {
description = "Specifies the number of days you want to retain log events for the lambda log group. Possible values are: 0, 1, 3, 5, 7, 14, 30, 60, 90, 120, 150, 180, 365, 400, 545, 731, 1827, and 3653."
type = number
default = 7
default = 180
}

variable "logging_kms_key_id" {
Expand Down
2 changes: 1 addition & 1 deletion modules/webhook/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ variable "role_path" {
variable "logging_retention_in_days" {
description = "Specifies the number of days you want to retain log events for the lambda log group. Possible values are: 0, 1, 3, 5, 7, 14, 30, 60, 90, 120, 150, 180, 365, 400, 545, 731, 1827, and 3653."
type = number
default = 7
default = 180
}

variable "logging_kms_key_id" {
Expand Down

0 comments on commit 86e6abc

Please sign in to comment.