Skip to content

Commit f8362d9

Browse files
sipr-invivozeritti
andauthored
[prometheus-pushgateway] remove default labels from persistent volume (#5280)
* [prometheus-pushgateway] fix: remove default labels from persistent volume Signed-off-by: sipr-invivo <160140834+sipr-invivo@users.noreply.github.com> * Bump chart version Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com> * Add upgrade note Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com> --------- Signed-off-by: sipr-invivo <160140834+sipr-invivo@users.noreply.github.com> Signed-off-by: zeritti <47476160+zeritti@users.noreply.github.com> Co-authored-by: zeritti <47476160+zeritti@users.noreply.github.com>
1 parent 3171fe0 commit f8362d9

File tree

3 files changed

+17
-2
lines changed

3 files changed

+17
-2
lines changed

charts/prometheus-pushgateway/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ apiVersion: v2
22
appVersion: "v1.11.0"
33
description: A Helm chart for prometheus pushgateway
44
name: prometheus-pushgateway
5-
version: 2.17.0
5+
version: 3.0.0
66
home: https://github.com/prometheus/pushgateway
77
sources:
88
- https://github.com/prometheus/pushgateway

charts/prometheus-pushgateway/README.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,19 @@ helm upgrade [RELEASE_NAME] prometheus-community/prometheus-pushgateway --instal
4141

4242
_See [helm upgrade](https://helm.sh/docs/helm/helm_upgrade/) for command documentation._
4343

44+
### To 3.0.0
45+
46+
Previously, as dynamic labels were also set on the statefulset's volume claim template, it was not possible
47+
to upgrade a chart release in a usual manner whilst using a statefulset and persistence due to the volume claim template's fields being immutable.
48+
49+
This release removes the dynamic labels from the statefulset's volume claim template.
50+
If you have configured a statefulset with persistent storage,
51+
please, delete the statefulset before upgrading:
52+
53+
```console
54+
kubectl delete sts -l app.kubernetes.io/name=prometheus-pushgateway --cascade=orphan
55+
```
56+
4457
### To 2.0.0
4558

4659
Chart API version has been upgraded to v2 so Helm 3 is needed from now on.

charts/prometheus-pushgateway/templates/statefulset.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,10 @@ spec:
2929
annotations:
3030
{{- toYaml . | nindent 10 }}
3131
{{- end }}
32+
{{- with .Values.persistentVolumeLabels }}
3233
labels:
33-
{{- include "prometheus-pushgateway.defaultLabels" . | nindent 10 }}
34+
{{- toYaml . | nindent 10 }}
35+
{{- end }}
3436
name: storage-volume
3537
spec:
3638
accessModes:

0 commit comments

Comments
 (0)