Cover scenario when connectionType is none for kubelogin in HelmDeployV0 (#18517)

* fix helmdeploy getAllPods undefined; make warning into error

* upgrade 0.224 to 0.224.1

Author: qianz2

Tasks/HelmDeployV0/src/helm.ts

@@ -78,17 +78,21 @@ function runHelmSaveCommand(helmCli: helmcli, kubectlCli: kubernetescli, failOnS
 
 async function run() {
     var command = tl.getInput("command", true).toLowerCase();
+    var connectionType = tl.getInput("connectionType", true);
     var isKubConfigRequired = isKubConfigSetupRequired(command);
     var kubectlCli: kubernetescli;
-    if (isKubConfigRequired) {
-        var kubeconfigfilePath = command === "logout" ? tl.getVariable("KUBECONFIG") : await getKubeConfigFile();
+    var externalAuth = connectionType === "None" && (command === "install" || command === "upgrade");
+    if (externalAuth && !tl.getVariable("KUBECONFIG")) {
+        tl.error("KUBECONFIG kube configuration file path must be set when connectionType is none and command is install or upgrade.");
+    }
+    if (isKubConfigRequired || externalAuth) {
+        var kubeconfigfilePath = (command === "logout" || externalAuth) ? tl.getVariable("KUBECONFIG") : await getKubeConfigFile();
         kubectlCli = new kubernetescli(kubeconfigfilePath);
         kubectlCli.login();
     }
 
     var helmCli: helmcli = new helmcli();
     helmCli.login();
-    var connectionType = tl.getInput("connectionType", true);
     var telemetry = {
         connectionType: connectionType,
         command: command,
@@ -120,7 +124,7 @@ async function run() {
         tl.setResult(tl.TaskResult.Failed, err.message);
     }
     finally {
-        if (isKubConfigLogoutRequired(command)) {
+        if (isKubConfigLogoutRequired(command) || externalAuth) {
             kubectlCli.logout();
         }
 
@@ -202,7 +206,7 @@ function runHelm(helmCli: helmcli, command: string, kubectlCli: kubernetescli, f
             }
         }
         catch (e) {
-            tl.warning("Capturing deployment metadata failed with error: " + e);
+            tl.error("Capturing deployment metadata failed with error: " + e);
         }
     }
 }

Tasks/HelmDeployV0/task.json

@@ -14,7 +14,7 @@
     "version": {
         "Major": 0,
         "Minor": 224,
-        "Patch": 0
+        "Patch": 1
     },
     "demands": [],
     "groups": [

Tasks/HelmDeployV0/task.loc.json

@@ -14,7 +14,7 @@
   "version": {
     "Major": 0,
     "Minor": 224,
-    "Patch": 0
+    "Patch": 1
   },
   "demands": [],
   "groups": [