Skip to content

Commit

Permalink
[stable/jasperreports] Fix chart not being upgradable (helm#7818)
Browse files Browse the repository at this point in the history
Signed-off-by: Javier J. Salmeron Garcia <jsalmeron@bitnami.com>
  • Loading branch information
javsalgar authored and wgiddens committed Jan 18, 2019
1 parent 4eceff5 commit 7691a71
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 1 deletion.
2 changes: 1 addition & 1 deletion stable/jasperreports/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
name: jasperreports
version: 2.0.4
version: 3.0.0
appVersion: 7.1.0
description: The JasperReports server can be used as a stand-alone or embedded reporting
and BI server that offers web-based reporting, analytic tools and visualization,
Expand Down
11 changes: 11 additions & 0 deletions stable/jasperreports/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -105,3 +105,14 @@ The [Bitnami JasperReports](https://github.com/bitnami/bitnami-docker-jasperrepo

Persistent Volume Claims are used to keep the data across deployments. This is known to work in GCE, AWS, and minikube.
See the [Configuration](#configuration) section to configure the PVC or to disable persistence.

## Upgrading

### To 3.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 3.0.0. The following example assumes that the release name is jasperreports:

```console
$ kubectl patch deployment jasperreports-jasperreports --type=json -p='[{"op": "remove", "path": "/spec/selector/matchLabels/chart"}]'
$ kubectl delete statefulset jasperreports-mariadb --cascade=false
4 changes: 4 additions & 0 deletions stable/jasperreports/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 "jasperreports.fullname" . }}
release: "{{ .Release.Name }}"
template:
metadata:
labels:
Expand Down

0 comments on commit 7691a71

Please sign in to comment.