Skip to content

infra: upgrade grafana loki#827

Merged
CallMeLeopoldo merged 3 commits into
masterfrom
ls/update-grafana-loki
Jan 15, 2026
Merged

infra: upgrade grafana loki#827
CallMeLeopoldo merged 3 commits into
masterfrom
ls/update-grafana-loki

Conversation

@CallMeLeopoldo

@CallMeLeopoldo CallMeLeopoldo commented Jan 15, 2026

Copy link
Copy Markdown
Contributor

The chart version was causing some issues as well as the config file that we were using for said release. The result of this change is the logs being displayed in Grafana (example was obtained by port-forwarding the grafana service and checking the datasource for loki)

image Screenshot 2026-01-15 at 12 06 47

@codacy-production

codacy-production Bot commented Jan 15, 2026

Copy link
Copy Markdown

Codacy's Analysis Summary

0 new issue (≤ 1 medium issue)
0 new security issue (≤ 0 issue)

Review Pull Request in Codacy →

AI Reviewer available: add the codacy-review label to get contextual insights without leaving GitHub.

@CallMeLeopoldo CallMeLeopoldo requested a review from a team as a code owner January 15, 2026 12:23

@jorgebraz jorgebraz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this change impact existing SH customers in any way?
For example: will customers have to change anything in their installations so logging works for them?

@CallMeLeopoldo

CallMeLeopoldo commented Jan 15, 2026

Copy link
Copy Markdown
Contributor Author

Does this change impact existing SH customers in any way? For example: will customers have to change anything in their installations so logging works for them?

Nope, this change only affects customers that are doing a fresh (and first) install of the SH instance.

@jorgebraz jorgebraz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚢

@CallMeLeopoldo CallMeLeopoldo merged commit 561f45e into master Jan 15, 2026
9 checks passed
@CallMeLeopoldo CallMeLeopoldo deleted the ls/update-grafana-loki branch January 15, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants