chore: remove unnecessary code and comments

Local mode checks were moved to `getForwardablePorts` in #5022.
garden-io · Sep 13, 2023 · 59d0ef3 · 59d0ef3
1 parent 668a246
commit 59d0ef3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/core/src/plugins/kubernetes/helm/deployment.ts b/core/src/plugins/kubernetes/helm/deployment.ts
@@ -147,7 +147,6 @@ export const helmDeploy: DeployActionHandler<"deploy", HelmDeployAction> = async
     timeoutSec: timeout,
   })
 
-  // Local mode has its own port-forwarding configuration
   const forwardablePorts = getForwardablePorts({ resources: manifests, parentAction: action, mode })
 
   // Make sure port forwards work after redeployment

diff --git a/core/src/plugins/kubernetes/kubernetes-type/handlers.ts b/core/src/plugins/kubernetes/kubernetes-type/handlers.ts
@@ -299,13 +299,10 @@ export const getKubernetesDeployStatus: DeployActionHandler<"getStatus", Kuberne
     }
   }
 
-  // Note: Local mode has its own port-forwarding configuration
-  if (deployedMode !== "local") {
-    try {
-      forwardablePorts = getForwardablePorts({ resources: remoteResources, parentAction: action, mode: deployedMode })
-    } catch (error) {
-      log.debug({ msg: `Unable to extract forwardable ports: ${error.message}`, error })
-    }
+  try {
+    forwardablePorts = getForwardablePorts({ resources: remoteResources, parentAction: action, mode: deployedMode })
+  } catch (error) {
+    log.debug({ msg: `Unable to extract forwardable ports: ${error.message}`, error })
   }
 
   return composeKubernetesDeployStatus({