update workflow

Author: Barsnes

.github/workflows/azure-preview-www.yml

@@ -52,66 +52,44 @@ jobs:
           push: true
           tags: ${{ env.ACR_NAME }}.azurecr.io/www:${{ env.APP_NAME }}-${{ github.sha }}
       - name: Create/Update Container App (Managed Identity)
-        env:
-          TIMEOUT_SECONDS: 300
-          SLEEP_SECONDS: 5
         run: |
           set -euo pipefail
-
-            if ! az containerapp show -n $APP_NAME -g $RESOURCE_GROUP >/dev/null 2>&1; then
-              echo "Creating container app $APP_NAME (temporary image)"
-              az containerapp create \
-                -n $APP_NAME \
-                -g $RESOURCE_GROUP \
-                --environment $CONTAINERAPPS_ENV \
-                --image mcr.microsoft.com/k8se/quickstart:latest \
-                --ingress external --target-port $PORT \
-                --env-vars PORT=$PORT HOST=0.0.0.0 \
-                --system-assigned
-            else
-              echo "Container app $APP_NAME exists"
+          if ! az containerapp show -n $APP_NAME -g $RESOURCE_GROUP >/dev/null 2>&1; then
+            echo "Creating container app $APP_NAME (temporary public image)"
+            az containerapp create \
+              -n $APP_NAME \
+              -g $RESOURCE_GROUP \
+              --environment $CONTAINERAPPS_ENV \
+              --image mcr.microsoft.com/k8se/quickstart:latest \
+              --ingress external --target-port $PORT \
+              --env-vars PORT=$PORT HOST=0.0.0.0 \
+              --system-assigned
+          else
+            echo "Container app $APP_NAME exists"
+            # Assign identity only if missing
+            PID=$(az containerapp show -n $APP_NAME -g $RESOURCE_GROUP --query identity.principalId -o tsv 2>/dev/null || true)
+            if [ -z "$PID" ] || [ "$PID" = "null" ]; then
+              echo "Adding system identity"
               az containerapp identity assign -n $APP_NAME -g $RESOURCE_GROUP --system-assigned >/dev/null || true
             fi
+          fi
 
           PRINCIPAL_ID=$(az containerapp show -n $APP_NAME -g $RESOURCE_GROUP --query identity.principalId -o tsv)
           ACR_ID=$(az acr show -n $ACR_NAME --query id -o tsv)
           echo "Ensuring AcrPull role (principal: $PRINCIPAL_ID) on $ACR_ID"
           az role assignment create --assignee $PRINCIPAL_ID --scope $ACR_ID --role AcrPull >/dev/null 2>&1 || echo "Role already assigned"
 
-          # Small wait to let RBAC settle
-          sleep 5
+          # Brief wait for RBAC propagation (kept minimal)
+          sleep 3
 
           echo "Setting registry (managed identity)"
           az containerapp registry set -n $APP_NAME -g $RESOURCE_GROUP --server $ACR_NAME.azurecr.io --identity system >/dev/null || true
 
           NEW_IMAGE="$ACR_NAME.azurecr.io/www:${APP_NAME}-${GITHUB_SHA}"
-          echo "Updating image -> $NEW_IMAGE"
-          az containerapp update -n $APP_NAME -g $RESOURCE_GROUP --image "$NEW_IMAGE" --set-env-vars PORT=$PORT HOST=0.0.0.0 --only-show-errors --no-wait
-
-          echo "Waiting for revision to become Healthy (timeout ${TIMEOUT_SECONDS}s)"
-          start_time=$(date +%s)
-          while true; do
-            # Break if timeout
-            now=$(date +%s)
-            elapsed=$(( now - start_time ))
-            if [ $elapsed -ge $TIMEOUT_SECONDS ]; then
-              echo "Timed out waiting for Healthy revision" >&2
-              az containerapp show -n $APP_NAME -g $RESOURCE_GROUP -o yaml || true
-              exit 1
-            fi
+          echo "Updating image -> $NEW_IMAGE (no wait)"
+          az containerapp update -n $APP_NAME -g $RESOURCE_GROUP --image "$NEW_IMAGE" --set-env-vars PORT=$PORT HOST=0.0.0.0 --only-show-errors --no-wait || true
 
-            REVISION=$(az containerapp revision list -n $APP_NAME -g $RESOURCE_GROUP --query "[?properties.active].name | [0]" -o tsv 2>/dev/null || true)
-            HEALTH=""
-            if [ -n "$REVISION" ]; then
-              HEALTH=$(az containerapp revision show -g $RESOURCE_GROUP --name $APP_NAME --revision $REVISION --query properties.healthState -o tsv 2>/dev/null || true)
-            fi
-            echo "Revision: ${REVISION:-<none>} Health: ${HEALTH:-<unknown>} (${elapsed}s)"
-            if [ "$HEALTH" = "Healthy" ]; then
-              echo "Revision $REVISION is Healthy"
-              break
-            fi
-            sleep $SLEEP_SECONDS
-          done
+          echo "Skipping wait for Healthy state; proceeding to comment with FQDN."
 
       - name: Fetch FQDN
         id: fqdn