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

Ignore opentelemetry logger in logging integration #3880

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

sl0thentr0py
Copy link
Member

Without this, internal otel logs (especially logger.exceptions) will show up as events / breadcrumbs in the payload.

Without this, internal otel logs (especially `logger.exception`s) will
show up as events / breadcrumbs in the payload.
Copy link

codecov bot commented Dec 18, 2024

❌ 282 Tests Failed:

Tests completed Failed Passed Skipped
13080 282 12798 3321
View the top 3 failed tests by shortest run time
 tests.integrations.celery.integration_tests.test_celery_beat_cron_monitoring
Stack Traces | 0s run time
No failure message available
 tests.integrations.celery.test_celery_beat_crons
Stack Traces | 0s run time
No failure message available
 tests.integrations.celery.test_update_celery_task_headers
Stack Traces | 0s run time
No failure message available

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@sl0thentr0py sl0thentr0py merged commit 7fca789 into potel-base Dec 18, 2024
98 of 125 checks passed
@sl0thentr0py sl0thentr0py deleted the neel/potel/ignore-logger branch December 18, 2024 12:00
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