Merge pull request #4526 from consideRatio/pr/bucket-tags

terraform, aws: support specifying bucket specific tags

terraform/aws/buckets.tf

@@ -1,7 +1,7 @@
 resource "aws_s3_bucket" "user_buckets" {
   for_each = var.user_buckets
   bucket   = lower("${var.cluster_name}-${each.key}")
-  tags     = var.tags
+  tags     = merge(var.tags, each.value.tags)
 }
 
 resource "aws_s3_bucket_lifecycle_configuration" "user_bucket_expiry" {

terraform/aws/variables.tf

@@ -23,7 +23,8 @@ variable "user_buckets" {
   type = map(
     object({
       delete_after : optional(number, null),
-      archival_storageclass_after : optional(number, null)
+      archival_storageclass_after : optional(number, null),
+      tags : optional(map(string), {}),
     })
   )
   default     = {}
@@ -40,6 +41,7 @@ variable "user_buckets" {
   2. `archival_storageclass_after` - number of days after *creation* an
      object in this bucket will be automatically transitioned to a cheaper,
      slower storageclass for cost savings. Set to null to not transition.
+  3. `tags` - bucket specific tags to be merged into the general tags variable.
   EOT
 }