Skip to content

Commit 1230d60

Browse files
Helm: Add revision history limit for worker daemonset (#1797)
* Helm: Add revision history limit for worker daemonset Signed-off-by: Rouke Broersma <[email protected]> * Helm: Add revision history limit for topology updater daemonset Signed-off-by: Rouke Broersma <[email protected]> * chore: tidy table columns --------- Signed-off-by: Rouke Broersma <[email protected]>
1 parent 876eea4 commit 1230d60

File tree

4 files changed

+11
-0
lines changed

4 files changed

+11
-0
lines changed

deployment/helm/node-feature-discovery/templates/topologyupdater.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ metadata:
1212
{{- toYaml . | nindent 4 }}
1313
{{- end }}
1414
spec:
15+
revisionHistoryLimit: {{ .Values.topologyUpdater.revisionHistoryLimit }}
1516
selector:
1617
matchLabels:
1718
{{- include "node-feature-discovery.selectorLabels" . | nindent 6 }}

deployment/helm/node-feature-discovery/templates/worker.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ metadata:
1212
{{- toYaml . | nindent 4 }}
1313
{{- end }}
1414
spec:
15+
revisionHistoryLimit: {{ .Values.worker.revisionHistoryLimit }}
1516
selector:
1617
matchLabels:
1718
{{- include "node-feature-discovery.selectorLabels" . | nindent 6 }}

deployment/helm/node-feature-discovery/values.yaml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -425,6 +425,9 @@ worker:
425425
# If not set and create is true, a name is generated using the fullname template
426426
name:
427427

428+
# specify how many old ControllerRevisions for the DaemonSet to retain.
429+
revisionHistoryLimit:
430+
428431
rbac:
429432
create: true
430433

@@ -468,6 +471,10 @@ topologyUpdater:
468471
create: true
469472
annotations: {}
470473
name:
474+
475+
# specify how many old ControllerRevisions for the DaemonSet to retain.
476+
revisionHistoryLimit:
477+
471478
rbac:
472479
create: true
473480

docs/deployment/helm.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,7 @@ API's you need to install the prometheus operator in your cluster.
168168
| `worker.annotations` | dict | {} | NFD worker pod [annotations](https://kubernetes.io/docs/concepts/overview/working-with-objects/annotations/) |
169169
| `worker.daemonsetAnnotations` | dict | {} | NFD worker daemonset [annotations](https://kubernetes.io/docs/concepts/overview/working-with-objects/annotations/) |
170170
| `worker.args` | array | [] | Additional [command line arguments](../reference/worker-commandline-reference.md) to pass to nfd-worker |
171+
| `worker.revisionHistoryLimit` | integer | | Specify how many old ControllerRevisions for this DaemonSet you want to retain. [revisionHistoryLimit](https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/daemon-set-v1/#DaemonSetSpec) |
171172

172173
### Topology updater parameters
173174

@@ -198,6 +199,7 @@ API's you need to install the prometheus operator in your cluster.
198199
| `topologyUpdater.podSetFingerprint` | bool | true | Enables compute and report of pod fingerprint in NRT objects. |
199200
| `topologyUpdater.kubeletStateDir` | string | /var/lib/kubelet | Specifies kubelet state directory path for watching state and checkpoint files. Empty value disables kubelet state tracking. |
200201
| `topologyUpdater.args` | array | [] | Additional [command line arguments](../reference/topology-updater-commandline-reference.md) to pass to nfd-topology-updater |
202+
| `topologyUpdater.revisionHistoryLimit` | integer | | Specify how many old ControllerRevisions for this DaemonSet you want to retain. [revisionHistoryLimit](https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/daemon-set-v1/#DaemonSetSpec) |
201203

202204
### Garbage collector parameters
203205

0 commit comments

Comments
 (0)