Merge pull request segmentio#73 from Edencia/s3-logs-expiration

achille-roussel · web-flow · commit 1bf32b34fce2 · 2017-04-24T10:48:10.000-07:00
S3 logs expiration
diff --git a/main.tf b/main.tf
@@ -139,6 +139,14 @@ variable "extra_cloud_config_content" {
   default     = ""
 }
 
+variable "logs_expiration_enabled" {
+  default = false
+}
+
+variable "logs_expiration_days" {
+  default = 30
+}
+
 module "defaults" {
   source = "./defaults"
   region = "${var.region}"
@@ -219,10 +227,12 @@ module "ecs_cluster" {
 }
 
 module "s3_logs" {
-  source      = "./s3-logs"
-  name        = "${var.name}"
-  environment = "${var.environment}"
-  account_id  = "${module.defaults.s3_logs_account_id}"
+  source                  = "./s3-logs"
+  name                    = "${var.name}"
+  environment             = "${var.environment}"
+  account_id              = "${module.defaults.s3_logs_account_id}"
+  logs_expiration_enabled = "${var.logs_expiration_enabled}"
+  logs_expiration_days    = "${var.logs_expiration_days}"
 }
 
 // The region in which the infra lives.
diff --git a/s3-logs/main.tf b/s3-logs/main.tf
@@ -7,6 +7,14 @@ variable "environment" {
 variable "account_id" {
 }
 
+variable "logs_expiration_enabled" {
+  default = false
+}
+
+variable "logs_expiration_days" {
+  default = 30
+}
+
 data "template_file" "policy" {
   template = "${file("${path.module}/policy.json")}"
 
@@ -19,6 +27,16 @@ data "template_file" "policy" {
 resource "aws_s3_bucket" "logs" {
   bucket = "${var.name}-${var.environment}-logs"
 
+  lifecycle_rule {
+    id = "logs-expiration"
+    prefix = ""
+    enabled = "${var.logs_expiration_enabled}"
+
+    expiration {
+      days = "${var.logs_expiration_days}"
+    }
+  }
+
   tags {
     Name        = "${var.name}-${var.environment}-logs"
     Environment = "${var.environment}"
