Skip to content

Commit ca4078c

Browse files
committed
chore(helm): add global clusterZone value
1 parent bd33041 commit ca4078c

5 files changed

Lines changed: 10 additions & 10 deletions

File tree

deploy/helm/CHANGELOG.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,11 @@
33
## [1.11.0] - 2026-06-30
44

55
### Added
6-
- `node.zone` Helm value for the node DaemonSet. **Required** with `--mode=node` to pass `--zone` to the CSI plugin
6+
- `clusterZone` value for both controller and node. Setting it avoids an API call on the controller startup, since it's needed for the DaemonSet already.
77

88
### Changed
9-
- Node DaemonSet runs `--mode=node` instead of `--mode=monolith`. UpCloud API credentials are no longer deployed to every cluster node, reducing credential blast radius
9+
- Node DaemonSet runs `--mode=node` now instead of `--mode=monolith`. UpCloud API credentials are no longer deployed to every cluster node, reducing credential blast radius
10+
- Controller StatefulSet and Node DaemonSet use `clusterZone` for the `--zone` flag
1011

1112
## [1.10.0] - 2026-06-28
1213

deploy/helm/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,4 +26,4 @@ annotations:
2626
url: https://github.com/upcloud-tools/upcloud-csi/issues
2727
artifacthub.io/changes: |
2828
- "Node DaemonSet switches to --mode=node — credentials no longer deployed to every node"
29-
- "Add node.zone value for the node DaemonSet — required with --mode=node"
29+
- "Add root-level clusterZone for both controller and node"

deploy/helm/templates/controller-statefulset.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ spec:
214214
- "--endpoint=$(CSI_ENDPOINT)"
215215
- "--nodehost=$(NODE_ID)"
216216
- "--mode=monolith"
217-
{{- with .Values.controller.zone }}
217+
{{- with .Values.clusterZone }}
218218
- "--zone={{ . }}"
219219
{{- end }}
220220
- "--address=tcp://0.0.0.0:13071"

deploy/helm/templates/node-daemonset.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ spec:
119119
- "--endpoint=$(CSI_ENDPOINT)"
120120
- "--nodehost=$(NODE_ID)"
121121
- "--mode=node"
122-
{{- with .Values.node.zone }}
122+
{{- with .Values.clusterZone }}
123123
- "--zone={{ . }}"
124124
{{- end }}
125125
- "--address=tcp://0.0.0.0:13071"

deploy/helm/values.yaml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ fullnameOverride: ""
77
# -- Common labels added to all resources.
88
commonLabels: {}
99

10+
# -- UpCloud zone (e.g. de-fra1). Passed to both controller and node pods.
11+
# The controller discovers it automatically if empty.
12+
clusterZone: "" # @schema type:string
13+
1014
# -- Default log verbosity, overridable per component.
1115
logLevel: 5
1216

@@ -37,8 +41,6 @@ image: # @schema additionalProperties: false
3741

3842
# Controller StatefulSet settings.
3943
controller: # @schema additionalProperties: false
40-
# -- UpCloud zone (e.g. de-fra1). Auto-detected from node hostname if empty.
41-
zone: "" # @schema type:string
4244
# -- Log verbosity (inherits global logLevel when null).
4345
logLevel: # @schema type:[integer, null]; minimum:0; maximum:10
4446

@@ -126,9 +128,6 @@ controller: # @schema additionalProperties: false
126128

127129
# Node DaemonSet settings.
128130
node: # @schema additionalProperties: false
129-
# -- UpCloud zone (e.g. de-fra1). Required for --mode=node.
130-
zone: "" # @schema type:string
131-
132131
# -- Log verbosity (inherits global logLevel when null).
133132
logLevel: # @schema type:[integer, null]; minimum:0; maximum:10
134133

0 commit comments

Comments
 (0)