fix: use correct manifest loglevel in the error message

garden-io · Sep 15, 2023 · 139b924 · 139b924
1 parent f5fb8f8
commit 139b924
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/core/src/plugins/kubernetes/api.ts b/core/src/plugins/kubernetes/api.ts
@@ -965,7 +965,7 @@ export const KUBECTL_RETRY_OPTS: RetryOpts = {
   maxRetries: 3,
   minTimeoutMs: 300,
   // forceRetry is important, because shouldRetry cannot handle ChildProcessError.
-  forceRetry: true
+  forceRetry: true,
 }
 
 export async function getKubeConfig(log: Log, ctx: PluginContext, provider: KubernetesProvider) {
@@ -985,7 +985,7 @@ export async function getKubeConfig(log: Log, ctx: PluginContext, provider: Kube
             log,
             args,
           }),
-          KUBECTL_RETRY_OPTS,
+        KUBECTL_RETRY_OPTS
       )
     }
     return load(kubeConfigStr)!

diff --git a/core/src/plugins/kubernetes/kubectl.ts b/core/src/plugins/kubernetes/kubectl.ts
@@ -109,7 +109,8 @@ export async function apply({
 
   const input = Buffer.from(encodeYamlMulti(manifests))
 
-  log.debug(`Applying Kubernetes manifests:\n${input.toString()}`)
+  const manifestLogLevel = "debug" as const
+  log[manifestLogLevel](`Applying Kubernetes manifests:\n${input.toString()}`)
 
   let args = ["apply"]
   dryRun && args.push("--dry-run")
@@ -128,7 +129,7 @@ export async function apply({
           args,
           input,
         }),
-        KUBECTL_RETRY_OPTS
+      KUBECTL_RETRY_OPTS
     )
   } catch (e) {
     if (e instanceof ChildProcessError) {
@@ -138,7 +139,7 @@ export async function apply({
 
           ${e.details.output}
 
-          Use the option "--log-level verbose" to see the kubernetes manifests that we attempted to apply through "kubectl apply".
+          Use the option "--log-level ${manifestLogLevel}" to see the kubernetes manifests that we attempted to apply through "kubectl apply".
           `,
       })
     }