Skip to content

Commit 070639b

Browse files
authored
[stable/redis-ha]: fix: replace deprecated gitversion (#348)
* [stable/redis-ha]: fix: replace deprecated gitversion * [stable/redis-ha]: fix: replace deprecated gitversion
1 parent 7569ac7 commit 070639b

File tree

4 files changed

+7
-5
lines changed

4 files changed

+7
-5
lines changed

charts/redis-ha/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ keywords:
55
- redis
66
- keyvalue
77
- database
8-
version: 4.34.3
8+
version: 4.34.4
99
appVersion: 8.2.1
1010
description: This Helm chart provides a highly available Redis implementation with a master/slave configuration and uses Sentinel sidecars for failover management
1111
icon: https://img.icons8.com/external-tal-revivo-shadow-tal-revivo/24/external-redis-an-in-memory-data-structure-project-implementing-a-distributed-logo-shadow-tal-revivo.png

charts/redis-ha/README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ This chart bootstraps a [Redis](https://redis.io) highly available master/slave
2222

2323
* Kubernetes 1.8+ with Beta APIs enabled
2424
* PV provisioner support in the underlying infrastructure
25+
* Helm v3+
2526

2627
## Upgrading the Chart
2728

@@ -492,4 +493,4 @@ This version introduced changes to the selector labels for the Deployments. Depe
492493
Error: UPGRADE FAILED: cannot patch "redis-redis-ha-haproxy" with kind Deployment: Deployment.apps "redis-redis-ha-haproxy" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app":"redis-ha-haproxy", "component":"haproxy", "release":"redis"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable
493494
```
494495

495-
In these cases, you'll need to delete the Deployment for the haproxy, or simple do a forceful upgrade.
496+
In these cases, you'll need to delete the Deployment for the haproxy, or simple do a forceful upgrade.

charts/redis-ha/README.md.gotmpl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ This chart bootstraps a [Redis](https://redis.io) highly available master/slave
2222

2323
* Kubernetes 1.8+ with Beta APIs enabled
2424
* PV provisioner support in the underlying infrastructure
25+
* Helm v3+
2526

2627
## Upgrading the Chart
2728

charts/redis-ha/templates/redis-ha-announce-service.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,21 +8,21 @@ apiVersion: v1
88
kind: Service
99
metadata:
1010
name: {{ $fullName }}-announce-{{ $i }}
11-
namespace: {{ $namespace | quote}}
11+
namespace: {{ $namespace | quote }}
1212
labels:
1313
{{ include "labels.standard" $root | indent 4 }}
1414
{{- range $key, $value := $root.Values.extraLabels }}
1515
{{ $key }}: {{ $value | quote }}
1616
{{- end }}
1717
annotations:
18-
{{- if (semverCompare "<=1.10-0" $.Capabilities.KubeVersion.GitVersion) }}
18+
{{- if (semverCompare "<=1.10-0" $.Capabilities.KubeVersion.Version) }}
1919
service.alpha.kubernetes.io/tolerate-unready-endpoints: "true"
2020
{{- end }}
2121
{{- if $root.Values.serviceAnnotations }}
2222
{{ toYaml $root.Values.serviceAnnotations | indent 4 }}
2323
{{- end }}
2424
spec:
25-
{{- if (semverCompare ">=1.11-0" $.Capabilities.KubeVersion.GitVersion) }}
25+
{{- if (semverCompare ">=1.11-0" $.Capabilities.KubeVersion.Version) }}
2626
publishNotReadyAddresses: true
2727
{{- end }}
2828
type: ClusterIP

0 commit comments

Comments
 (0)