fix: set log retention in each module by default to 180 (#3433)

npalm · web-flow · commit 86e6abc174d7 · 2023-08-16T09:35:36.000+02:00
Top level module has set log retention to 180 days. This PR's ensure
that each module is applying the same default.
diff --git a/modules/multi-runner/variables.tf b/modules/multi-runner/variables.tf
@@ -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" {
diff --git a/modules/runner-binaries-syncer/variables.tf b/modules/runner-binaries-syncer/variables.tf
@@ -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" {
diff --git a/modules/webhook/variables.tf b/modules/webhook/variables.tf
@@ -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" {

Original file line number	Diff line number	Diff line change
`@@ -224,7 +224,7 @@ variable "role_path" {`
`224`	`224`	`variable "logging_retention_in_days" {`
`225`	`225`	`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."`
`226`	`226`	`type = number`
`227`		`- default = 7`
	`227`	`+ default = 180`
`228`	`228`	`}`
`229`	`229`
`230`	`230`	`variable "logging_kms_key_id" {`
Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ variable "runner_architecture" {`
`119`	`119`	`variable "logging_retention_in_days" {`
`120`	`120`	`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."`
`121`	`121`	`type = number`
`122`		`- default = 7`
	`122`	`+ default = 180`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`variable "logging_kms_key_id" {`
Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ variable "role_path" {`
`80`	`80`	`variable "logging_retention_in_days" {`
`81`	`81`	`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."`
`82`	`82`	`type = number`
`83`		`- default = 7`
	`83`	`+ default = 180`
`84`	`84`	`}`
`85`	`85`
`86`	`86`	`variable "logging_kms_key_id" {`