-
Notifications
You must be signed in to change notification settings - Fork 167
Open
Description
was walking through the driver-install instructions and ran into errors that prevented the controllers from starting:
kubectl logs -n gce-pd-csi-driver csi-gce-pd-node-6m6c5 -c gce-pd-driver | head -10
flag provided but not defined: -enable-multitenancy
Usage of /gce-pd-csi-driver:
-add_dir_header
If true, adds the file directory to the header of the log messages
-allow-hdha-provisioning
If set to true, will allow the driver to provision Hyperdisk-balanced High Availability disks
-alsologtostderr
log to standard error as well as files (no effect when -logtostderr=true)
-attach-disk-backoff-cap duration
our friend gemini mentioned this was no longer supported and suggested deleting the flag lines and I did in controller.yaml and node.yaml and things applied fine and started up and I was able to apply demo-zonal-sc.yaml
and its pod via demo-pod.yaml
just fine. the diff for my edits was:
git diff
diff --git a/deploy/kubernetes/base/controller/controller.yaml b/deploy/kubernetes/base/controller/controller.yaml
index 302874f8..14bdc793 100644
--- a/deploy/kubernetes/base/controller/controller.yaml
+++ b/deploy/kubernetes/base/controller/controller.yaml
@@ -146,7 +146,7 @@ spec:
- "--supports-dynamic-throughput-provisioning=hyperdisk-balanced,hyperdisk-throughput,hyperdisk-ml"
- --enable-data-cache
- --run-node-service=false
- - --enable-multitenancy
+ # - --enable-multitenancy
command:
- /gce-pd-csi-driver
env:
diff --git a/deploy/kubernetes/base/node_linux/node.yaml b/deploy/kubernetes/base/node_linux/node.yaml
index ef9e89bc..970b5565 100644
--- a/deploy/kubernetes/base/node_linux/node.yaml
+++ b/deploy/kubernetes/base/node_linux/node.yaml
@@ -47,7 +47,7 @@ spec:
- "--endpoint=unix:/csi/csi.sock"
- "--run-controller-service=false"
- "--enable-data-cache"
- - "--enable-multitenancy"
+ # - "--enable-multitenancy"
- "--node-name=$(KUBE_NODE_NAME)"
securityContext:
privileged: true
all of the above was on the master
branch just now.
git log -1
commit 623471408aa65dd5d07295f3b282d6faf27b9700 (HEAD -> master, origin/master, origin/HEAD)
Merge: d285089d 772b8d7b
Author: Matt Cary <[email protected]>
Date: Mon Oct 6 13:11:05 2025 -0700
Merge pull request #2186 from julianKatz/constants-in-own-package
Move constants.go into new pkg/constants
Metadata
Metadata
Assignees
Labels
No labels