Skip to content

feat(python): add DD_TRACE_ATEXIT_ENABLED to config normalization rules#6653

Merged
ZStriker19 merged 2 commits intomainfrom
zach.groves/python-atexit-config
Apr 1, 2026
Merged

feat(python): add DD_TRACE_ATEXIT_ENABLED to config normalization rules#6653
ZStriker19 merged 2 commits intomainfrom
zach.groves/python-atexit-config

Conversation

@ZStriker19
Copy link
Copy Markdown
Contributor

@ZStriker19 ZStriker19 commented Mar 31, 2026

Summary

  • Adds DD_TRACE_ATEXIT_ENABLED to python_config_rules.json normalization rules
  • Fixes test_config_telemetry_completeness failure caused by dd-trace-py/dd-trace-py#17195

DataDog/dd-trace-py#17195

Context

dd-trace-py#17195 adds DD_TRACE_ATEXIT_ENABLED (a Python-only config). Since _get_config from the telemetry module auto-registers every config it reads, the new key appears in telemetry payloads and needs a normalization rule here.

Normalized name: trace_atexit_enabled (follows existing DD_TRACE_*trace_* pattern in this file).

@github-actions
Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

utils/telemetry/intake/static/python_config_rules.json                  @DataDog/apm-sdk

@ZStriker19 ZStriker19 marked this pull request as ready for review March 31, 2026 16:55
@ZStriker19 ZStriker19 requested a review from a team as a code owner March 31, 2026 16:55
@ZStriker19 ZStriker19 requested review from zacharycmontoya and removed request for a team March 31, 2026 16:55
@datadog-prod-us1-3
Copy link
Copy Markdown

datadog-prod-us1-3 bot commented Mar 31, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 8ddd6a1 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

@ZStriker19 ZStriker19 enabled auto-merge (squash) March 31, 2026 17:49
@ZStriker19 ZStriker19 merged commit a97fef1 into main Apr 1, 2026
439 checks passed
@ZStriker19 ZStriker19 deleted the zach.groves/python-atexit-config branch April 1, 2026 14:32
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