Merge pull request #30 from clouddrove/yaac

added retention_in_days

Author: Om Sharma

_example/redis/example.tf

@@ -58,6 +58,7 @@ module "redis" {
   availability_zones         = ["eu-west-1a", "eu-west-1b"]
   auto_minor_version_upgrade = true
   number_cache_clusters      = 2
+  retention_in_days          = 0
 
   log_delivery_configuration = [
     {

main.tf

@@ -21,9 +21,10 @@ module "labels" {
 }
 
 resource "aws_cloudwatch_log_group" "default" {
-  count = var.enable && length(var.log_delivery_configuration) > 0 ? 1 : 0
-  name  = format("logs-%s", module.labels.id)
-  tags  = module.labels.tags
+  count             = var.enable && length(var.log_delivery_configuration) > 0 ? 1 : 0
+  name              = format("logs-%s", module.labels.id)
+  retention_in_days = var.retention_in_days
+  tags              = module.labels.tags
 }
 
 # Module      : Elasticache Subnet Group

variables.tf

@@ -241,3 +241,9 @@ variable "log_delivery_configuration" {
   default     = []
   description = "The log_delivery_configuration block allows the streaming of Redis SLOWLOG or Redis Engine Log to CloudWatch Logs or Kinesis Data Firehose. Max of 2 blocks."
 }
+
+variable "retention_in_days" {
+  type        = number
+  default     = 0
+  description = "Specifies the number of days you want to retain log events in the specified log group."
+}