Skip to content

Commit

Permalink
Merge pull request #803 from bitnami/memcachedFixChart
Browse files Browse the repository at this point in the history
[bitnami/memcached] Fix chart not being upgradable
  • Loading branch information
juan131 authored Sep 24, 2018
2 parents f38da43 + 7689bfc commit af9398d
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 1 deletion.
2 changes: 1 addition & 1 deletion bitnami/memcached/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: memcached
version: 0.4.35
version: 1.0.0
appVersion: 1.5.10
description: Chart for Memcached
keywords:
Expand Down
11 changes: 11 additions & 0 deletions bitnami/memcached/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,3 +72,14 @@ $ helm install --name my-release -f values.yaml bitnami/memcached
```

> **Tip**: You can use the default [values.yaml](values.yaml)
## Upgrading

### To 1.0.0

Backwards compatibility is not guaranteed unless you modify the labels used on the chart's deployments.
Use the workaround below to upgrade from versions previous to 1.0.0. The following example assumes that the release name is memcached:

```console
$ kubectl patch deployment memcached --type=json -p='[{"op": "remove", "path": "/spec/selector/matchLabels/chart"}]'
```
4 changes: 4 additions & 0 deletions bitnami/memcached/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@ metadata:
release: "{{ .Release.Name }}"
heritage: "{{ .Release.Service }}"
spec:
selector:
matchLabels:
app: {{ template "fullname" . }}
release: "{{ .Release.Name }}"
template:
metadata:
labels:
Expand Down

0 comments on commit af9398d

Please sign in to comment.