Address code review feedback: improve variable descriptions and fix ACR module

Co-authored-by: sayedimac <25403967+sayedimac@users.noreply.github.com>

Author: Copilot

azd/aca/variables.tf

@@ -32,12 +32,12 @@ variable "container_port" {
 
 variable "cpu_cores" {
   type        = string
-  description = "CPU cores allocated to the container"
+  description = "CPU cores allocated to the container (e.g., '0.5', '1.0')"
   default     = "0.5"
 }
 
 variable "memory_in_gb" {
   type        = string
-  description = "Memory allocated to the container in GB"
+  description = "Memory allocated to the container (e.g., '1Gi', '2Gi')"
   default     = "1Gi"
 }

azd/aci/variables.tf

@@ -32,12 +32,12 @@ variable "container_port" {
 
 variable "cpu_cores" {
   type        = string
-  description = "CPU cores allocated to the container instance"
+  description = "CPU cores allocated to the container instance (e.g., '0.5', '1.0', '2.0')"
   default     = "1.0"
 }
 
 variable "memory_in_gb" {
   type        = string
-  description = "Memory allocated to the container instance in GB"
+  description = "Memory allocated to the container instance in GB (e.g., '1.0', '1.5', '2.0')"
   default     = "1.5"
 }

azd/acr/modules/acr.tf

@@ -30,13 +30,11 @@ variable "registry_name" {
 variable "location" {
   type        = string
   description = "Azure region location"
-  default     = null
 }
 
 variable "resource_group_name" {
   type        = string
   description = "Name of the resource group"
-  default     = null
 }
 
 variable "sku" {

azd/aks/main.tf

@@ -43,6 +43,7 @@ resource "azurerm_container_registry" "acr" {
 }
 
 # AKS Cluster
+# Note: RBAC is disabled to match the Bicep configuration, but enabling RBAC is recommended for production
 resource "azurerm_kubernetes_cluster" "aks" {
   name                              = "aks-${random_string.resource_token.result}"
   location                          = azurerm_resource_group.main.location