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

Add support for Sentry monitoring #33

Merged
merged 3 commits into from
Dec 6, 2024
Merged

Add support for Sentry monitoring #33

merged 3 commits into from
Dec 6, 2024

Conversation

pvannierop
Copy link
Contributor

This PR will implement platform observability/monitoring with Sentry. Because of the version of the Confluent base image, we need to use the legacy log4j 1.x implementation. When the SENTRY_DSN environmental variable is not set, the log4j appender is not activated and logs are not sent to Sentry. Includes documentation.

@pvannierop pvannierop requested a review from keyvaann December 5, 2024 11:30
@pvannierop pvannierop self-assigned this Dec 5, 2024
Copy link
Contributor

@keyvaann keyvaann left a comment

Choose a reason for hiding this comment

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

LGTM

@pvannierop pvannierop force-pushed the feature/sentry branch 4 times, most recently from ea1feb7 to 83421d1 Compare December 5, 2024 11:48
Includes addition to documentation.
@pvannierop pvannierop force-pushed the feature/sentry branch 2 times, most recently from 9985d04 to de96052 Compare December 5, 2024 13:29
@keyvaann keyvaann merged commit 48343f0 into master Dec 6, 2024
3 of 5 checks passed
@keyvaann keyvaann deleted the feature/sentry branch December 6, 2024 09:09
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