Merge pull request #79 from dminkovski/aks

Updated exit code to 0 on eaysauth scripts

Author: dantelmomsft

.devcontainer/devcontainer.json

@@ -18,17 +18,15 @@
         "ghcr.io/devcontainers/features/git:1": {
             "version": "2.39.1"
         },
-        "ghcr.io/devcontainers-contrib/features/typescript:2": {
-            
-        }
-        ,
+        "ghcr.io/devcontainers-contrib/features/typescript:2": {},
+        "ghcr.io/devcontainers/features/kubectl-helm-minikube:1": {
+            "version": "1.1.8"
+        },
         "docker-in-docker": {
             "version": "latest",
             "moby": true,
             "dockerDashComposeVersion": "v1"
-        },
-        "ghcr.io/devcontainers/features/kubectl-helm-minikube:1": {}
-    
+        }
     },
     "customizations": {
         "vscode": {

deploy/aks/scripts/easyauth-down.ps1

@@ -13,7 +13,7 @@ foreach ($line in $output) {
 if($Env:AZURE_USE_EASY_AUTH -eq "true"){
   Write-Host "Enabled EasyAuth for the AKS Cluster"
 } else {
-  exit 1;
+  exit 0;
 }
 
 Write-Host "Deleting Entra App ${Env:adAppName}:${Env:AZURE_AD_APP_ID}"

deploy/aks/scripts/easyauth-down.sh

@@ -22,7 +22,7 @@ done <<< "$output"
 if [ "$AZURE_USE_EASY_AUTH" == "true" ]; then
   echo "Enabled EasyAuth for the AKS Cluster"
 else
-  exit 1
+  exit 0
 fi
 
 # Deleting Entra App

deploy/aks/scripts/easyauth.ps1

@@ -19,7 +19,7 @@ if($env:AZURE_USE_EASY_AUTH -eq "true"){
 } else {
   Write-Host "EasyAuth is not enabled for the AKS Cluster"
   Write-Host "If you want to enable EasyAuth, please set the AZURE_USE_EASY_AUTH environment variable to true"
-  exit 1;
+  exit 0;
 }
 
 $location = $env:AZURE_LOCATION

deploy/aks/scripts/easyauth.sh

@@ -37,7 +37,7 @@ if [ "$AZURE_USE_EASY_AUTH" = "true" ]; then
 else
   echo "EasyAuth is not enabled for the AKS Cluster"
   echo "If you want to enable EasyAuth, please set the AZURE_USE_EASY_AUTH environment variable to true"
-  exit 1
+  exit 0
 fi
 
 location=$AZURE_LOCATION