Skip to content

Commit

Permalink
[stable/redis-ha] Fix label selector of haproxy servicemonitor (helm#…
Browse files Browse the repository at this point in the history
…20209)

* [stable/redis-ha] Fix label selector of haproxy servicemonitor

The old servicemonitor did not actually match the deployed service,
because the service uses redis-ha.fullname as a label
(https://github.com/helm/charts/blob/master/stable/redis-ha/templates/redis-haproxy-service.yaml#L9)
but the servicemonitor matches redis-ha.name.

Signed-off-by: Max Rosin <m.rosin@syseleven.de>

* Raise chart version after rebase

Signed-off-by: Max Rosin <git@hackrid.de>
  • Loading branch information
ekeih authored and includerandom committed Jul 19, 2020
1 parent d70f292 commit 8e54a23
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion stable/redis-ha/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ keywords:
- redis
- keyvalue
- database
version: 4.4.0
version: 4.4.1
appVersion: 5.0.6
description: Highly available Kubernetes implementation of Redis
icon: https://upload.wikimedia.org/wikipedia/en/thumb/6/6b/Redis_Logo.svg/1200px-Redis_Logo.svg.png
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,5 +30,5 @@ spec:
matchLabels:
app: {{ template "redis-ha.name" . }}
release: {{ .Release.Name }}
component: {{ template "redis-ha.name" . }}-haproxy
component: {{ template "redis-ha.fullname" . }}-haproxy
{{- end }}

0 comments on commit 8e54a23

Please sign in to comment.