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.

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

Update monitoring guide as dependencies are outdated #215

Open
nicolopicasso opened this issue Apr 6, 2024 · 4 comments
Open

Update monitoring guide as dependencies are outdated #215

nicolopicasso opened this issue Apr 6, 2024 · 4 comments
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation help wanted Extra attention is needed

Comments

@nicolopicasso
Copy link

New versions of Kubernetes can't run the charts anymore.

Upgrading charts version is not enough to make it work since prometheus StatefulSet crashes (kube-prometheus-stack:57.2.1)

Tested on AWS-powered cluster created with eksctl folllowing AWS setup guide.

@J12934
Copy link
Member

J12934 commented Apr 6, 2024

Hi @nicolopicasso
thank you for pointing this out 🙌

Could you open up a PR to increment the version numbers?

@J12934 J12934 added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file help wanted Extra attention is needed labels Apr 6, 2024
@rtfmkiesel
Copy link

@J12934 Just updating the numbers is not enough. As OP said, the monitoring charts are broken right now.

From my testing:

  • kube-prometheus-stack 57.2.1 works
    • web UI accessible
    • Juiceshop panel loads but has no data obv.
  • promtail 6.5.15 works
    • pod starts fine, didn't do more tests
  • loki 6.5.0 does not work:
Error: template: loki/templates/write/statefulset-write.yaml:50:28: executing "loki/templates/write/statefulset-write.yaml" at <include (print .Template.BasePath "/config.yaml")
...
error calling include: template: loki/templates/_helpers.tpl:228:19: executing "loki.commonStorageConfig" at <$.Values.loki.storage.bucketNames.chunks>: nil pointer evaluating interface {}.chunks

@ontanj
Copy link

ontanj commented Aug 20, 2024

any progress on this?

@PandaFood
Copy link

A Loki config is needed for the new version. I've made a PR for bumping the versions and adding the config #258

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants