do not enable glacier transition by default (#20)

* make glacier transition optional
closes #17
relates to cloudposse/terraform-aws-s3-log-storage#23

* do not enable glacier transition by default

* Updated README.md

* Executed 'terraform fmt'

Co-authored-by: Maxim Mironenko <simixido@gmail.com>
Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>

README.md

@@ -130,6 +130,7 @@ Available targets:
 | acl | The canned ACL to apply. We recommend log-delivery-write for compatibility with AWS services | string | `log-delivery-write` | no |
 | attributes | Additional attributes (e.g. `1`) | list(string) | `<list>` | no |
 | delimiter | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes` | string | `-` | no |
+| enable_glacier_transition | Glacier transition might just increase your bill. Set to false to disable lifecycle transitions to AWS Glacier. | bool | `false` | no |
 | enabled | Set to `false` to prevent the module from creating any resources | bool | `true` | no |
 | environment | Environment, e.g. 'prod', 'staging', 'dev', 'pre-prod', 'UAT' | string | `` | no |
 | expiration_days | Number of days after which to expunge the objects | string | `90` | no |

docs/terraform.md

@@ -5,6 +5,7 @@
 | acl | The canned ACL to apply. We recommend log-delivery-write for compatibility with AWS services | string | `log-delivery-write` | no |
 | attributes | Additional attributes (e.g. `1`) | list(string) | `<list>` | no |
 | delimiter | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes` | string | `-` | no |
+| enable_glacier_transition | Glacier transition might just increase your bill. Set to false to disable lifecycle transitions to AWS Glacier. | bool | `false` | no |
 | enabled | Set to `false` to prevent the module from creating any resources | bool | `true` | no |
 | environment | Environment, e.g. 'prod', 'staging', 'dev', 'pre-prod', 'UAT' | string | `` | no |
 | expiration_days | Number of days after which to expunge the objects | string | `90` | no |

main.tf

@@ -58,7 +58,7 @@ data "aws_iam_policy_document" "default" {
 }
 
 module "s3_bucket" {
-  source                             = "git::https://github.com/cloudposse/terraform-aws-s3-log-storage.git?ref=tags/0.7.0"
+  source                             = "git::https://github.com/cloudposse/terraform-aws-s3-log-storage.git?ref=tags/0.8.0"
   enabled                            = var.enabled
   namespace                          = var.namespace
   stage                              = var.stage
@@ -76,6 +76,7 @@ module "s3_bucket" {
   noncurrent_version_transition_days = var.noncurrent_version_transition_days
   standard_transition_days           = var.standard_transition_days
   glacier_transition_days            = var.glacier_transition_days
+  enable_glacier_transition          = var.enable_glacier_transition
   expiration_days                    = var.expiration_days
   sse_algorithm                      = var.sse_algorithm
   kms_master_key_arn                 = var.kms_master_key_arn

variables.tf

@@ -114,6 +114,12 @@ variable "glacier_transition_days" {
   default     = 60
 }
 
+variable "enable_glacier_transition" {
+  type        = bool
+  default     = false
+  description = "Glacier transition might just increase your bill. Set to false to disable lifecycle transitions to AWS Glacier."
+}
+
 variable "expiration_days" {
   description = "Number of days after which to expunge the objects"
   default     = 90