use map instead of set

damianj · damianj · commit 86ff682935e4 · 2025-11-06T14:27:56.000-06:00
diff --git a/infra/app-flask/service/bda.tf b/infra/app-flask/service/bda.tf
@@ -40,6 +40,9 @@ module "bda_test" {
       tags                   = [{ key = "example_tag_key", value = "example_tag_value" }]
     }
   }
-  name_prefix        = "bda_module_test"
-  bucket_policy_arns = toset([module.bda_input_bucket.access_policy_arn, module.bda_output_bucket.access_policy_arn])
+  name_prefix = "bda_module_test"
+  bucket_policy_arns = {
+    input_bucket  = module.bda_input_bucket.access_policy_arn,
+    output_bucket = module.bda_output_bucket.access_policy_arn
+  }
 }
diff --git a/infra/modules/bedrock-data-automation/variables.tf b/infra/modules/bedrock-data-automation/variables.tf
@@ -5,7 +5,7 @@ variable "name_prefix" {
 
 variable "bucket_policy_arns" {
   description = "The set of policy ARNs for the input and output buckets to attach to the BDA role."
-  type        = set(string)
+  type        = map(string)
 }
 
 variable "bda_project_description" {

Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,9 @@ module "bda_test" {`
`40`	`40`	`tags = [{ key = "example_tag_key", value = "example_tag_value" }]`
`41`	`41`	`}`
`42`	`42`	`}`
`43`		`- name_prefix = "bda_module_test"`
`44`		`- bucket_policy_arns = toset([module.bda_input_bucket.access_policy_arn, module.bda_output_bucket.access_policy_arn])`
	`43`	`+ name_prefix = "bda_module_test"`
	`44`	`+ bucket_policy_arns = {`
	`45`	`+ input_bucket = module.bda_input_bucket.access_policy_arn,`
	`46`	`+ output_bucket = module.bda_output_bucket.access_policy_arn`
	`47`	`+ }`
`45`	`48`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ variable "name_prefix" {`
`5`	`5`
`6`	`6`	`variable "bucket_policy_arns" {`
`7`	`7`	`description = "The set of policy ARNs for the input and output buckets to attach to the BDA role."`
`8`		`- type = set(string)`
	`8`	`+ type = map(string)`
`9`	`9`	`}`
`10`	`10`
`11`	`11`	`variable "bda_project_description" {`