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

Removal of Liveness Probe in Loki 3.x for Query Frontend #13967

Open
kashish1200 opened this issue Aug 27, 2024 · 3 comments · May be fixed by #15359
Open

Removal of Liveness Probe in Loki 3.x for Query Frontend #13967

kashish1200 opened this issue Aug 27, 2024 · 3 comments · May be fixed by #15359
Labels
area/helm

Comments

@kashish1200
Copy link

kashish1200 commented Aug 27, 2024 •
edited
Loading

I’ve been reviewing the Loki 3.x configuration, and I noticed that the liveness probe is no longer present for certain components, specifically the Query Frontend . In previous versions, these components did have liveness probes configured.

Could you please provide some insight into why the liveness probes were removed for these components in Loki 3.x?
@chaudum
Copy link
Contributor

chaudum commented Aug 27, 2024

@kashish1200 Is this question regarding the Helm chart?

@kashish1200
Copy link
Author

@chaudum Yes, my question is related to the Helm chart.

@chaudum
Copy link
Contributor

chaudum commented Aug 29, 2024

Could you please provide some insight into why the liveness probes were removed for these components in Loki 3.x?

I don't think that there were particular reasons for removing them, they probably got forgotten about in the rewrite of the Helm chart for 3.0

This was referenced Dec 11, 2024
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/helm
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(helm): Add health probes to long-lived containers lindhe/loki
2 participants
@chaudum @kashish1200