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

feat: log stream limit errors back to insights logging #15163

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cstyan
Copy link
Contributor

@cstyan cstyan commented Nov 27, 2024

At the moment stream_limit, as in the limit for the maximum # of active streams a tenant may have, is only propagated back to end users via HTTP errors to their logs agent. We have internal logging of these errors as well, but it would be useful to also propagate such error messages back to Grafana Cloud Logs users in their Logs Usage Insights datasource so that they can more easily self serve investigations into these errors, and so we could also build a dashboard to aid in such investigations if we decided to do so.

This PR adds logging of the stream_limit errors to the writeFailures logging, which is just a simple wrapper for correctly logging the tenant ID and insight=true to ingestion path logging which we want to propagated to usage insights datasources.

Signed-off-by: Callum Styan <callumstyan@gmail.com>
@cstyan cstyan requested a review from a team as a code owner November 27, 2024 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants