Skip to content

chore(tracer): disable telemetry logs for monkey errors#17227

Open
brettlangdon wants to merge 1 commit intomainfrom
dd/fix-monkey-telemetry-log-opt-out
Open

chore(tracer): disable telemetry logs for monkey errors#17227
brettlangdon wants to merge 1 commit intomainfrom
dd/fix-monkey-telemetry-log-opt-out

Conversation

@brettlangdon
Copy link
Copy Markdown
Member

@brettlangdon brettlangdon commented Mar 31, 2026

Description

This change prevents two expected integration patching error logs in ddtrace/_monkey.py from being forwarded as telemetry error logs, while keeping them at error log level locally.

  • Updated both log.error(...) calls inside _on_import_factory exception handlers in ddtrace/_monkey.py to include extra={"send_to_telemetry": False}.
  • Preserved existing telemetry integration failure reporting via telemetry.telemetry_writer.add_integration(...) and integration_errors metric emission.
  • Motivation: reduce noisy telemetry log ingestion for these handled integration patch failures without losing integration health/error signal.

Co-authored-by: brettlangdon <me@brett.is>
@datadog-official
Copy link
Copy Markdown
Contributor

View session in Datadog

Bits Dev status: ✅ Done

CI Auto-fix: Disabled | Enable

Comment @DataDog to request changes

@brettlangdon brettlangdon requested a review from a team as a code owner March 31, 2026 16:59
@brettlangdon brettlangdon requested a review from juanjux March 31, 2026 16:59
@datadog-prod-us1-4
Copy link
Copy Markdown

I can only run on private repositories.

@cit-pr-commenter-54b7da
Copy link
Copy Markdown

Codeowners resolved as

ddtrace/_monkey.py                                                      @DataDog/apm-core-python

@brettlangdon brettlangdon added the changelog/no-changelog A changelog entry is not required for this PR. label Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bits AI changelog/no-changelog A changelog entry is not required for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant