-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
🌱 hack/observability: Bump Loki, Grafana and promtail charts #7182
🌱 hack/observability: Bump Loki, Grafana and promtail charts #7182
Conversation
452bae4
to
6f90ec9
Compare
756c3e5
to
c086a5d
Compare
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale We should still bump when the chart is fixed |
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs. This bot triages PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale Currently trying to improve the Loki Helm chart so we can finally finish this |
48a3ea1
to
3f986f1
Compare
/assign @killianmuldoon @fabriziopandini @ykakarap I did a quick local test with logs and metrics it seems to be working. EDIT: Also verified log push now. Would be great if someone can double check. |
Signed-off-by: Stefan Büringer [email protected]
3f986f1
to
8f91bc6
Compare
Btw I think we should continue pinning Helm chart versions. I think it's the only safe thing to do. |
This is great! |
LGTM label has been added. Git tree hash: 4f61636966dab1dc4b71c5087a2aa2a446b62aee
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/area devtools |
What this PR does / why we need it:
This PR bumps Loki, Grafana and promtail Helm charts
just for reference, we had to resolve the following issues to unblock this change:
loki.auth_enabled
to false (it's true per default)Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #