Skip to content

Commit

Permalink
[stable/grafana] Update sidecar image configuration (helm#22801)
Browse files Browse the repository at this point in the history
This change makes the configuration of the sidecar image consistent with
the other images used by the chart. This change is important because it
allows you to change the repository while leaving the version untouched
which is desirable when you consume the chart via a repo manager, such
as Artifactory.

Signed-off-by: Reinhard Nägele <unguiculus@gmail.com>
  • Loading branch information
unguiculus authored Jun 16, 2020
1 parent 11964fb commit 1cfefc9
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion stable/grafana/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v1
name: grafana
version: 5.1.4
version: 5.2.0
appVersion: 7.0.3
kubeVersion: "^1.8.0-0"
description: The leading tool for querying and visualizing time series and metrics.
Expand Down
3 changes: 2 additions & 1 deletion stable/grafana/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,8 @@ You have to add --force to your helm upgrade command as the labels of the chart
| `podAnnotations` | Pod annotations | `{}` |
| `podLabels` | Pod labels | `{}` |
| `podPortName` | Name of the grafana port on the pod | `grafana` |
| `sidecar.image` | Sidecar image | `kiwigrid/k8s-sidecar:0.1.151` |
| `sidecar.image.repository` | Sidecar image repository | `kiwigrid/k8s-sidecar` |
| `sidecar.image.tag` | Sidecar image tag | `0.1.151` |
| `sidecar.imagePullPolicy` | Sidecar image pull policy | `IfNotPresent` |
| `sidecar.resources` | Sidecar resources | `{}` |
| `sidecar.dashboards.enabled` | Enables the cluster wide search for dashboards and adds/updates/deletes them in grafana | `false` |
Expand Down
4 changes: 2 additions & 2 deletions stable/grafana/templates/_pod.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ initContainers:
{{- end }}
{{- if .Values.sidecar.datasources.enabled }}
- name: {{ template "grafana.name" . }}-sc-datasources
image: "{{ .Values.sidecar.image }}"
image: "{{ .Values.sidecar.image.repository }}:{{ .Values.sidecar.image.tag }}"
imagePullPolicy: {{ .Values.sidecar.imagePullPolicy }}
env:
- name: METHOD
Expand Down Expand Up @@ -96,7 +96,7 @@ imagePullSecrets:
containers:
{{- if .Values.sidecar.dashboards.enabled }}
- name: {{ template "grafana.name" . }}-sc-dashboard
image: "{{ .Values.sidecar.image }}"
image: "{{ .Values.sidecar.image.repository }}:{{ .Values.sidecar.image.tag }}"
imagePullPolicy: {{ .Values.sidecar.imagePullPolicy }}
env:
- name: METHOD
Expand Down
4 changes: 3 additions & 1 deletion stable/grafana/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -464,7 +464,9 @@ smtp:
## Sidecars that collect the configmaps with specified label and stores the included files them into the respective folders
## Requires at least Grafana 5 to work and can't be used together with parameters dashboardProviders, datasources and dashboards
sidecar:
image: kiwigrid/k8s-sidecar:0.1.151
image:
repository: kiwigrid/k8s-sidecar
tag: 0.1.151
imagePullPolicy: IfNotPresent
resources: {}
# limits:
Expand Down

0 comments on commit 1cfefc9

Please sign in to comment.