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

DEV-47164 - Add alerting observability in logs #68

Merged

Conversation

yasmin-tr
Copy link

What is this feature?

We need more logs data to be able to investigate alerting issues.
Main data missing is to correlate on a single evaluation all the logs - that requires rule uid, and eval time.

Also removed some debug log that were excessive and changed log levels in some places.

Why do we need this feature?

Better observability and monitoring on grafana alerting.

@yasmin-tr
Copy link
Author

run-test

1 similar comment
@npololnskii
Copy link

run-test

@yasmin-tr
Copy link
Author

run-test

@yasmin-tr
Copy link
Author

run-test

5 similar comments
@yasmin-tr
Copy link
Author

run-test

@yasmin-tr
Copy link
Author

run-test

@yasmin-tr
Copy link
Author

run-test

@yasmin-tr
Copy link
Author

run-test

@yasmin-tr
Copy link
Author

run-test

@yasmin-tr yasmin-tr force-pushed the DEV-47164-add-alerting-observability-in-logs branch from 20c5a0a to 1b21242 Compare November 25, 2024 08:26
@yasmin-tr
Copy link
Author

run-test

@yasmin-tr
Copy link
Author

run-test

Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update or ping for review. Thank you for your contributions!

@github-actions github-actions bot added the stale label Dec 27, 2024
@yasmin-tr
Copy link
Author

run-test

@yasmin-tr yasmin-tr merged commit f0130ac into v10.4.x-logzio Jan 5, 2025
14 of 17 checks passed
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.

2 participants