From 007fe55a7b8a9aac2270fd49feca59b40a0e22c0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Carlos=20Rodr=C3=ADguez=20Hern=C3=A1ndez?= Date: Tue, 11 Feb 2020 19:00:43 +0100 Subject: [PATCH] [stable/redis] Adapt docs to Helm 3 (#20677) * [stable/redis] Adapt docs to Helm 3 Signed-off-by: Carlos Rodriguez Hernandez * Update README Signed-off-by: Carlos Rodriguez Hernandez --- stable/redis/Chart.yaml | 2 +- stable/redis/README.md | 16 ++++++++-------- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/stable/redis/Chart.yaml b/stable/redis/Chart.yaml index 574f3bba35..47944e2242 100644 --- a/stable/redis/Chart.yaml +++ b/stable/redis/Chart.yaml @@ -1,6 +1,6 @@ apiVersion: v1 name: redis -version: 10.4.3 +version: 10.4.4 appVersion: 5.0.7 description: Open source, advanced key-value store. It is often referred to as a data structure server since keys can contain strings, hashes, lists, sets and sorted sets. keywords: diff --git a/stable/redis/README.md b/stable/redis/README.md index 448d46dcb5..ee60a01eb3 100644 --- a/stable/redis/README.md +++ b/stable/redis/README.md @@ -7,12 +7,12 @@ ```bash # Testing configuration -$ helm install stable/redis +$ helm install my-release stable/redis ``` ```bash # Production configuration -$ helm install stable/redis --values values-production.yaml +$ helm install my-release stable/redis --values values-production.yaml ``` ## Introduction @@ -32,7 +32,7 @@ Bitnami charts can be used with [Kubeapps](https://kubeapps.com/) for deployment To install the chart with the release name `my-release`: ```bash -$ helm install --name my-release stable/redis +$ helm install my-release stable/redis ``` The command deploys Redis on the Kubernetes cluster in the default configuration. The [Parameters](#parameters) section lists the parameters that can be configured during installation. @@ -246,7 +246,7 @@ The following table lists the configurable parameters of the Redis chart and the Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`. For example, ```bash -$ helm install --name my-release \ +$ helm install my-release \ --set password=secretpassword \ stable/redis ``` @@ -256,7 +256,7 @@ The above command sets the Redis server password to `secretpassword`. Alternatively, a YAML file that specifies the values for the parameters can be provided while installing the chart. For example, ```bash -$ helm install --name my-release -f values.yaml stable/redis +$ helm install my-release -f values.yaml stable/redis ``` > **Tip**: You can use the default [values.yaml](values.yaml) @@ -376,7 +376,7 @@ By default, the chart mounts a [Persistent Volume](http://kubernetes.io/docs/use 3. Install the chart ```bash -$ helm install --set persistence.existingClaim=PVC_NAME stable/redis +$ helm install my-release --set persistence.existingClaim=PVC_NAME stable/redis ``` ## NetworkPolicy @@ -429,14 +429,14 @@ This version causes a change in the Redis Master StatefulSet definition, so the - Recommended: Create a clone of the Redis Master PVC (for example, using projects like [this one](https://github.com/edseymour/pvc-transfer)). Then launch a fresh release reusing this cloned PVC. ``` - helm install stable/redis --set persistence.existingClaim= + helm install my-release stable/redis --set persistence.existingClaim= ``` - Alternative (not recommended, do at your own risk): `helm delete --purge` does not remove the PVC assigned to the Redis Master StatefulSet. As a consequence, the following commands can be done to upgrade the release ``` helm delete --purge - helm install stable/redis --name + helm install stable/redis ``` Previous versions of the chart were not using persistence in the slaves, so this upgrade would add it to them. Another important change is that no values are inherited from master to slaves. For example, in 6.0.0 `slaves.readinessProbe.periodSeconds`, if empty, would be set to `master.readinessProbe.periodSeconds`. This approach lacked transparency and was difficult to maintain. From now on, all the slave parameters must be configured just as it is done with the masters.