remove comment about tags since tag filters are not supported in S3 Express lifecycle configurations

magreenbaum · magreenbaum · commit 249fc87789a6 · 2025-02-01T11:38:33.000-05:00
diff --git a/modules/directory-bucket/main.tf b/modules/directory-bucket/main.tf
@@ -61,7 +61,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "this" {
         }
       }
 
-      # Max 1 block - filter - with one key argument or a single tag
+      # Max 1 block - filter - with one key argument
       dynamic "filter" {
         for_each = [for v in try(flatten([rule.value.filter]), []) : v if max(length(keys(v))) == 1]
 
@@ -72,7 +72,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "this" {
         }
       }
 
-      # Max 1 block - filter - with more than one key arguments or multiple tags
+      # Max 1 block - filter - with more than one key arguments
       dynamic "filter" {
         for_each = [for v in try(flatten([rule.value.filter]), []) : v if max(length(keys(v))) > 1]
 

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "this" {`
`61`	`61`	`}`
`62`	`62`	`}`
`63`	`63`
`64`		`- # Max 1 block - filter - with one key argument or a single tag`
	`64`	`+ # Max 1 block - filter - with one key argument`
`65`	`65`	`dynamic "filter" {`
`66`	`66`	`for_each = [for v in try(flatten([rule.value.filter]), []) : v if max(length(keys(v))) == 1]`
`67`	`67`
`@@ -72,7 +72,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "this" {`
`72`	`72`	`}`
`73`	`73`	`}`
`74`	`74`
`75`		`- # Max 1 block - filter - with more than one key arguments or multiple tags`
	`75`	`+ # Max 1 block - filter - with more than one key arguments`
`76`	`76`	`dynamic "filter" {`
`77`	`77`	`for_each = [for v in try(flatten([rule.value.filter]), []) : v if max(length(keys(v))) > 1]`
`78`	`78`