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

Configuration plans for Loki charm #283

Closed
Abuelodelanada opened this issue Jun 30, 2023 · 2 comments · Fixed by #325
Closed

Configuration plans for Loki charm #283

Abuelodelanada opened this issue Jun 30, 2023 · 2 comments · Fixed by #325

Comments

@Abuelodelanada
Copy link
Contributor

Enhancement Proposal

Read this post:

The context is that after instrumenting my machine charm with grafana-agent and setting up a local COS-lite environment, I was able to see logs in Grafana and proceeded to setup a few panels that displayed logs for different sources - syslog, model logs, unit logs, etc. Mainly to observe what the difference was between them. That’s when I ran into a bit of an issue, my Grafana panels all timed out and displayed an error “too many outstanding requests”.

Further investigation landed me on a few threads - like - this suggesting changes to Loki config.

The Loki charm currently doesn’t offer much in the way of config so my question is how will users of the COS-lite stack be able to tune their deployment. My understanding is that sensible charm defaults should get one most of the way there but predictably there might be a need to configure parameters for large and small scale operations.

@Abuelodelanada
Copy link
Contributor Author

@sed-i said:

Alertmanager has the simplest config yaml of them all and we ended up with providing it in its entirety via config option, rather than deconstructing to reconstruct.

See this config option

Is this possible in Loki? Is there any other option?

@lucabello
Copy link
Contributor

The root issue should be addressed by incrementing the default value of the max_concurrent config; see this comment for reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants