Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[prometheus-kube-stack] Update grafana helm chart dependency #876

Closed
wasfree opened this issue Apr 22, 2021 · 1 comment · Fixed by #877
Closed

[prometheus-kube-stack] Update grafana helm chart dependency #876

wasfree opened this issue Apr 22, 2021 · 1 comment · Fixed by #877
Labels
bug Something isn't working

Comments

@wasfree
Copy link
Contributor

wasfree commented Apr 22, 2021

Describe the bug
Currently kube-prometheus-stack has a dependency to grafana helm chart version 6.7.4. This Version has and issue when generating the ConfigMap for grafana.ini, lines longer than 120 chars will be wrapped causing an invalid yaml file.

YAML parse error on kube-prometheus-stack/charts/grafana/templates/configmap.yaml: error converting YAML to JSON: yaml: line 45: could not find expected ':'

Fortunately this issue was aleady fixed in Version 6.7.5 grafana/helm-charts#386. However latest version is 6.8.0 so it would make sense to update dependency to grafana to this version.
@wasfree wasfree added the bug Something isn't working label Apr 22, 2021
@wasfree wasfree mentioned this issue Apr 22, 2021
Merged
3 tasks
@ArchiFleKs
Copy link
Contributor

I also cannot use ingress on grafana on the latest version : 15.1.3 maybe because of this

bismarck pushed a commit that referenced this issue Apr 22, 2021
#876 Update kube-prometheus-stack grafana dependency to 6.8.0 (#877)
531ce5c 
Signed-off-by: Nick Metz <nick.metz@ptvgroup.com>
langecode pushed a commit to neticdk/helm-charts that referenced this issue Aug 12, 2021
@langecode
prometheus-community#876 Update kube-prometheus-stack grafana depende…
bf8e8fc 
…ncy to 6.8.0 (prometheus-community#877)

Signed-off-by: Nick Metz <nick.metz@ptvgroup.com>
Signed-off-by: Thor Anker Kvisgård Lange <tal@netic.dk>
stamzid pushed a commit to Unstructured-IO/prometheus-community-helm-charts that referenced this issue Mar 3, 2023
prometheus-community#876 Update kube-prometheus-stack grafana depende…
3cd0af6 
…ncy to 6.8.0 (prometheus-community#877)

Signed-off-by: Nick Metz <nick.metz@ptvgroup.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[kube-prometheus-stack] #876 Update kube-prometheus-stack grafana dependency to 6.8.0
2 participants
@ArchiFleKs @wasfree