made access point config optional

cloudacademy · Apr 22, 2024 · 8d8533c · 8d8533c
1 parent 6534cda
commit 8d8533c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/terraform/modules/efs/main.tf b/terraform/modules/efs/main.tf
@@ -13,7 +13,7 @@ resource "aws_efs_file_system" "file_system" {
 }
 
 resource "aws_efs_access_point" "access_point" {
-  count = var.posix_access_point_config == {} || var.root_access_point_config == {} ? 0 : 1
+  count = var.posix_access_point_config == null || var.root_access_point_config == null ? 0 : 1
 
   file_system_id = aws_efs_file_system.file_system.id
 

diff --git a/terraform/modules/efs/vars.tf b/terraform/modules/efs/vars.tf
@@ -23,19 +23,19 @@ variable "CORE_BACKUPS_RETENTION" {
 variable "posix_access_point_config" {
   description = "The configuration for the POSIX access point"
   type = object({
-    posix_user_gid = optional(string)
-    posix_user_uid = optional(string)
+    posix_user_gid = string
+    posix_user_uid = string
   })
-  default = {}
+  default = null
 }
 
 variable "root_access_point_config" {
   description = "The configuration for the root access point"
   type = object({
-    path        = optional(string)
-    owner_gid   = optional(string)
-    owner_uid   = optional(string)
-    permissions = optional(string)
+    path        = string
+    owner_gid   = string
+    owner_uid   = string
+    permissions = string
   })
-  default = {}
+  default = null
 }