variables.tf (7 lines): - line 56: #TODO - verify what the access policy name represents and the scope of the policy name. Is this a set of defaults or does it include any custom policies and do they have to be in the same RG or sub? - line 89: #TODO - Can we review this with the PG to determine if they intend to improve the target representation? (single value CIDR syntax as an option?) - line 196: #TODO: Determine the valid linking hierarchies. We will create linkages assuming this instance is the primary. - line 395: #TODO come back to this and see if Enterprise has similar configuration blocks that can be plumbed up here. Confirm if we need to mark this as sensitive - line 408: data_persistence_authentication_method = optional(string) #TODO: research the managed identity vs. SAS key and determine level of effort required to default to ManagedIdentity as the more secure option, and review what happens if data persistence is not enabled. - line 409: rdb_backup_enabled = optional(bool) #TODO: Research if we want backups to be true. Given this is cache, probably not required. - line 515: description = "(Required) - The Redis SKU to use. Possible values are `Basic`, `Standard`, and `Premium`. Note: Downgrading the sku will force new resource creation." #TODO validate whether we can merge Open Source and Premium skus