testing

Signed-off-by: sk593 <shruthikumar@microsoft.com>

Author: sk593

.github/scripts/configure-azure-provider.sh

@@ -62,7 +62,8 @@ fi
 
 printf "\033[34;1m=>\033[0m Configuring Azure provider for Radius tests\n"
 
-if ! az group exists --name "$AZURE_RESOURCE_GROUP" --subscription "$AZURE_SUBSCRIPTION_ID" >/dev/null 2>&1; then
+# az group exists returns "true" or "false" as text, not an exit code
+if [[ "$(az group exists --name "$AZURE_RESOURCE_GROUP" --subscription "$AZURE_SUBSCRIPTION_ID" 2>/dev/null)" != "true" ]]; then
     echo "Error: Azure resource group '$AZURE_RESOURCE_GROUP' not found. Create it before running configure-azure-provider." >&2
     exit 1
 fi

.github/workflows/validate-azure-recipes.yaml

@@ -18,11 +18,64 @@ permissions:
   contents: read
 
 jobs:
+  setup-azure:
+    runs-on: ubuntu-24.04
+    timeout-minutes: 10
+    name: Setup Azure Resource Group
+    environment: azure
+    permissions:
+      id-token: write
+      contents: read
+    outputs:
+      resource-group: ${{ steps.set-context.outputs.resource-group }}
+      location: ${{ steps.set-context.outputs.location }}
+    steps:
+      - name: Azure Login
+        uses: azure/login@a457da9ea143d694b1b9c7c869ebb04ebe844ef5 # v2.3.0
+        with:
+          client-id: ${{ secrets.AZURE_CLIENT_ID }}
+          tenant-id: ${{ secrets.AZURE_TENANT_ID }}
+          subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
+
+      - name: Set Azure Test Context
+        id: set-context
+        run: |
+          LOCATION="${AZURE_LOCATION:-${{ vars.AZURE_LOCATION }}}"
+          if [ -z "$LOCATION" ]; then
+            LOCATION="westus3"
+          fi
+          RG="rrttest-${{ github.run_id }}-${{ github.run_attempt }}"
+          echo "location=$LOCATION" >> "$GITHUB_OUTPUT"
+          echo "resource-group=$RG" >> "$GITHUB_OUTPUT"
+
+      - name: Create Azure Resource Group
+        env:
+          AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
+        run: |
+          set -euo pipefail
+          RG="${{ steps.set-context.outputs.resource-group }}"
+          LOCATION="${{ steps.set-context.outputs.location }}"
+          current_time=$(date +%s)
+          az group create \
+            --only-show-errors \
+            --output none \
+            --location "$LOCATION" \
+            --name "$RG" \
+            --subscription "$AZURE_SUBSCRIPTION_ID" \
+            --tags creationTime=$current_time > /dev/null
+          # Wait for resource group to be fully available
+          while [[ "$(az group exists --name "$RG" --subscription "$AZURE_SUBSCRIPTION_ID")" != "true" ]]; do
+            echo "Waiting for resource group '$RG' to be available..."
+            sleep 5
+          done
+          echo "Resource group '$RG' is ready"
+
   validate-azure-recipes:
     runs-on: ubuntu-24.04
     timeout-minutes: 30
     name: Validate Azure ${{ matrix.recipe }} Recipes
     environment: azure
+    needs: setup-azure
     strategy:
       fail-fast: false
       matrix:
@@ -34,6 +87,8 @@ jobs:
       AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }}
       AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }}
       AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
+      AZURE_RESOURCE_GROUP: ${{ needs.setup-azure.outputs.resource-group }}
+      AZURE_LOCATION: ${{ needs.setup-azure.outputs.location }}
     steps:
       - name: Checkout
         uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
@@ -49,31 +104,11 @@ jobs:
 
       - name: Set Azure Test Context
         run: |
-          LOCATION="${AZURE_LOCATION:-${{ vars.AZURE_LOCATION }}}"
-          if [ -z "$LOCATION" ]; then
-            LOCATION="westus3"
-          fi
-          RG="rrttest-${{ github.run_id }}-${{ github.run_attempt }}"
-          echo "AZURE_LOCATION=$LOCATION" >> "$GITHUB_ENV"
-          echo "AZURE_RESOURCE_GROUP=$RG" >> "$GITHUB_ENV"
+          echo "AZURE_LOCATION=$AZURE_LOCATION" >> "$GITHUB_ENV"
+          echo "AZURE_RESOURCE_GROUP=$AZURE_RESOURCE_GROUP" >> "$GITHUB_ENV"
           echo "AZURE_WORKSPACE_NAME=default" >> "$GITHUB_ENV"
           echo "AZURE_ENVIRONMENT_NAME=default" >> "$GITHUB_ENV"
 
-      - name: Create Azure Resource Group
-        run: |
-          set -euo pipefail
-          current_time=$(date +%s)
-          az group create \
-            --only-show-errors \
-            --output none \
-            --location "$AZURE_LOCATION" \
-            --name "$AZURE_RESOURCE_GROUP" \
-            --subscription "$AZURE_SUBSCRIPTION_ID" \
-            --tags creationTime=$current_time > /dev/null
-          while ! az group exists --name "$AZURE_RESOURCE_GROUP" --subscription "$AZURE_SUBSCRIPTION_ID" &>/dev/null; do
-            sleep 2
-          done
-
       - name: Setup Node
         uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
         with:
@@ -132,10 +167,35 @@ jobs:
         if: always()
         uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5.0.0
         with:
-          name: radius-pod-logs
+          name: radius-pod-logs-${{ matrix.recipe }}
           path: radius-pod-logs
           if-no-files-found: warn
 
+  cleanup-azure:
+    runs-on: ubuntu-24.04
+    timeout-minutes: 10
+    name: Cleanup Azure Resource Group
+    environment: azure
+    needs: [setup-azure, validate-azure-recipes]
+    if: always()
+    permissions:
+      id-token: write
+      contents: read
+    env:
+      AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
+      AZURE_RESOURCE_GROUP: ${{ needs.setup-azure.outputs.resource-group }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
+        with:
+          persist-credentials: false
+
+      - name: Azure Login
+        uses: azure/login@a457da9ea143d694b1b9c7c869ebb04ebe844ef5 # v2.3.0
+        with:
+          client-id: ${{ secrets.AZURE_CLIENT_ID }}
+          tenant-id: ${{ secrets.AZURE_TENANT_ID }}
+          subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
+
       - name: Cleanup Azure Resources
-        if: always()
         run: make cleanup-azure-resources

Compute/containers/recipes/azure/terraform/main.tf

@@ -75,11 +75,12 @@ locals {
       try(local.connection_definitions[conn_name].disableDefaultEnvVars, false) != true
     ? [
       # Add resource properties directly from connection (excluding metadata properties)
+      # Only include scalar values (strings, numbers, bools) - skip objects/lists
       for prop_name, prop_value in conn : {
         name  = upper("CONNECTION_${conn_name}_${prop_name}")
         value = tostring(prop_value)
       }
-      if !contains(local.excluded_properties, prop_name)
+      if !contains(local.excluded_properties, prop_name) && can(tostring(prop_value))
     ]
     : []
   ])