fix(helm): handle missing (null) values in version check (#5307)

* fix(helm): handle missing (null) values in version check * chore: improve log message Co-authored-by: Anna Mager <[email protected]> * chore: more precise deployment status handling * chore: print log statement in the proper place --------- Co-authored-by: Anna Mager <[email protected]>
garden-io · Oct 27, 2023 · 31f4420 · 31f4420
1 parent 6c18b6d
commit 31f4420
Showing 1 changed file with 21 additions and 15 deletions.
diff --git a/core/src/plugins/kubernetes/helm/status.ts b/core/src/plugins/kubernetes/helm/status.ts
@@ -205,22 +205,28 @@ export async function getReleaseStatus({
 
     if (state === "ready") {
       // Make sure the right version is deployed
-      values = JSON.parse(
-        await helm({
-          ctx,
-          log,
-          namespace,
-          args: ["get", "values", releaseName, "--output", "json"],
-          // do not send JSON output to Garden Cloud or CLI verbose log
-          emitLogEvents: false,
-        })
-      )
-
-      const deployedVersion = values[".garden"]?.version
-      deployedMode = values[".garden"]?.mode
-
-      if (action.mode() !== deployedMode || !deployedVersion || deployedVersion !== action.versionString()) {
+      const helmResponse = await helm({
+        ctx,
+        log,
+        namespace,
+        args: ["get", "values", releaseName, "--output", "json"],
+        // do not send JSON output to Garden Cloud or CLI verbose log
+        emitLogEvents: false,
+      })
+      values = JSON.parse(helmResponse)
+
+      let deployedVersion: string | undefined = undefined
+      // JSON.parse can return null
+      if (values === null) {
+        log.verbose(`No helm values returned for release ${releaseName}. Is this release managed outside of garden?`)
         state = "outdated"
+      } else {
+        deployedVersion = values[".garden"]?.version
+        deployedMode = values[".garden"]?.mode
+
+        if (action.mode() !== deployedMode || !deployedVersion || deployedVersion !== action.versionString()) {
+          state = "outdated"
+        }
       }
 
       // If ctx.cloudApi is defined, the user is logged in and they might be trying to deploy to an environment