Skip to content

Commit 74e97ee

Browse files
committed
[smokeping-prober] explicitly set serviceAccountName: default
0.3.0 dropped the SA template but left serviceAccountName unset, which Helm 3-way merge preserves from the previous release — pods then fail because the (now deleted) SA can't be found. Set it explicitly to "default" so the merge overwrites the stale field. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Signed-off-by: Alex Vorona <alex@vorona.com.ua>
1 parent c6e9b33 commit 74e97ee

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

dysnix/smokeping-prober/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v2
22
name: smokeping-prober
33
description: Prometheus style smokeping exporter
44
type: application
5-
version: 0.3.0
5+
version: 0.3.1
66
appVersion: "v0.11.0"
77
keywords:
88
- monitoring

dysnix/smokeping-prober/templates/daemonset.yaml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,11 @@ spec:
3333
{{- toYaml . | nindent 8 }}
3434
{{- end }}
3535
spec:
36-
## smokeping_prober has no Kubernetes API client — skip the SA token mount.
36+
## smokeping_prober has no Kubernetes API client. Use the namespace
37+
## default SA explicitly (so Helm 3-way merge overwrites any prior
38+
## serviceAccountName set by older chart versions) and skip the token
39+
## mount.
40+
serviceAccountName: default
3741
automountServiceAccountToken: false
3842
{{- with .Values.imagePullSecrets }}
3943
imagePullSecrets:

0 commit comments

Comments
 (0)