CloudNationHQ / terraform-azure-sa

Terraform module which creates storage resources used by workloads and accelerators.
https://library.tf/modules/CloudNationHQ/sa/azure/latest
MIT License
1 stars 2 forks source link

[BUG] - ACL's for each share should be allowed to have multiple #97

Closed eddy-vera closed 3 months ago

eddy-vera commented 3 months ago

Is there an existing issue for this?

Module Version

0.23.0

Terraform Version

1.8.1

AzureRM Provider Version

3.114.0

Short Description

ACL's for each share should be allowed to have multiple, use a map of objects instead of list of objects with one single acl

Terraform Configuration Files

 shares = {
        fs1 = {
          quota = 50
          metadata = {
            environment = "dev"
            owner       = "finance team"
          }
          acl = {
            acl1 = {
              access_policy = {
                permissions = "r"
                start       = "2024-07-02T09:38:21.0000000Z"
                expiry      = "2025-07-02T10:38:21.0000000Z"
              }
            }
            acl2 = {
              access_policy = {
                permissions = "rwdl"
                start       = "2024-08-01T09:38:21.0000000Z"
                expiry      = "2025-08-01T10:38:21.0000000Z"
              }
            }
          }
        }
      }

Expected Behaviour

Created multiple ACL's (two in this case).

Actual Behaviour

Got an error (see below), as the first element of the list was referenced, instead of multiple acl entries. │ The argument "acl.0.id" is required, but no definition was found.

Steps to Reproduce

No response

Additional Context

No response

References

No response