fix: controller doesn't need mode helm value

Author: aardbol

deploy/helm/CHANGELOG.md

@@ -3,7 +3,7 @@
 ## [1.11.0] - 2026-06-30
 
 ### Added
-- `node.zone` Helm value with fallback to `controller.zone`. Required for node DaemonSet to pass `--zone` to `--mode=node`
+- `node.zone` Helm value for the node DaemonSet. **Required** with `--mode=node` to pass `--zone` to the CSI plugin
 
 ### Changed
 - Node DaemonSet runs `--mode=node` instead of `--mode=monolith`. UpCloud API credentials are no longer deployed to every cluster node, reducing credential blast radius

deploy/helm/Chart.yaml

@@ -26,4 +26,4 @@ annotations:
       url: https://github.com/upcloud-tools/upcloud-csi/issues
   artifacthub.io/changes: |
     - "Node DaemonSet switches to --mode=node — credentials no longer deployed to every node"
-    - "Add node.zone value with fallback to controller.zone — required for --mode=node to start correctly"
+    - "Add node.zone value for the node DaemonSet — required with --mode=node"

deploy/helm/templates/node-daemonset.yaml

@@ -119,9 +119,8 @@ spec:
             - "--endpoint=$(CSI_ENDPOINT)"
             - "--nodehost=$(NODE_ID)"
             - "--mode=node"
-            {{- $zone := .Values.node.zone | default .Values.controller.zone }}
-            {{- if $zone }}
-            - "--zone={{ $zone }}"
+            {{- with .Values.node.zone }}
+            - "--zone={{ . }}"
             {{- end }}
             - "--address=tcp://0.0.0.0:13071"
             {{- if .Values.metrics.enabled }}

deploy/helm/values.yaml

@@ -126,7 +126,7 @@ controller:  # @schema additionalProperties: false
 
 # Node DaemonSet settings.
 node:  # @schema additionalProperties: false
-  # -- UpCloud zone (e.g. de-fra1). Required for --mode=node; falls back to controller.zone if empty.
+  # -- UpCloud zone (e.g. de-fra1). Required for --mode=node.
   zone: ""  # @schema type:string
 
   # -- Log verbosity (inherits global logLevel when null).