fix(k8s): don't install NFS provisioner when sync storage class is set

garden-service/src/plugins/kubernetes/init.ts

@@ -201,14 +201,27 @@ export async function prepareSystem(
 
   await installTiller({ ctx: sysCtx, provider: sysCtx.provider, log, force })
 
+  // We need to install the NFS provisioner separately, so that we can optionally install it
+  // FIXME: when we've added an `enabled` field, we should get rid of this special case
+  if (systemServiceNames.includes("nfs-provisioner")) {
+    await prepareSystemServices({
+      log,
+      sysGarden,
+      namespace: systemNamespace,
+      force,
+      ctx: k8sCtx,
+      serviceNames: ["nfs-provisioner"],
+    })
+  }
+
   // Install system services
   await prepareSystemServices({
     log,
     sysGarden,
     namespace: systemNamespace,
     force,
     ctx: k8sCtx,
-    serviceNames: systemServiceNames,
+    serviceNames: systemServiceNames.filter(name => name !== "nfs-provisioner"),
   })
 
   sysGarden.log.setSuccess()

garden-service/static/kubernetes/system/build-sync/garden.yml

@@ -3,8 +3,10 @@ type: helm
 name: build-sync
 description: Sync service for receiving build context ahead of in-cluster builds
 releaseName: garden-build-sync
-dependencies:
-  - nfs-provisioner
+# Note: We special-case this dependency in the cluster-init procedure,
+# to be fixed when the `enabled` field is implemented
+# dependencies:
+#   - nfs-provisioner
 values:
   resources:
     limits: