Skip to content

chore: runtime version string in ddtrace_serverless#17964

Merged
emmettbutler merged 12 commits into
mainfrom
emmett.butler/name-metadata-match
May 12, 2026
Merged

chore: runtime version string in ddtrace_serverless#17964
emmettbutler merged 12 commits into
mainfrom
emmett.butler/name-metadata-match

Conversation

@emmettbutler
Copy link
Copy Markdown
Collaborator

@emmettbutler emmettbutler commented May 7, 2026

This change handles the fact that __package__ and importlib.metadata.version(__package__) don't always match at runtime, specifically in the ddtrace_serverless wheel.

@emmettbutler emmettbutler requested review from a team as code owners May 7, 2026 18:56
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 42e4286199

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread ddtrace/version.py Outdated
@cit-pr-commenter-54b7da
Copy link
Copy Markdown

cit-pr-commenter-54b7da Bot commented May 7, 2026

Codeowners resolved as

benchmarks/suitespec.yml                                                @DataDog/apm-core-python
ddtrace/version.py                                                      @DataDog/python-guild @DataDog/apm-core-python

@DataDog DataDog deleted a comment from chatgpt-codex-connector Bot May 7, 2026
Co-authored-by: Brett Langdon <brett.langdon@datadoghq.com>
@emmettbutler emmettbutler added the changelog/no-changelog A changelog entry is not required for this PR. label May 7, 2026
@emmettbutler emmettbutler changed the title fix: runtime version string in ddtrace_serverless chore: runtime version string in ddtrace_serverless May 7, 2026
@emmettbutler emmettbutler added changelog/no-changelog A changelog entry is not required for this PR. and removed changelog/no-changelog A changelog entry is not required for this PR. labels May 7, 2026
@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented May 7, 2026

Benchmarks

Benchmark execution time: 2026-05-12 14:18:22

Comparing candidate commit 8d1ec22 in PR branch emmett.butler/name-metadata-match with baseline commit c03c39c in branch main.

Found 0 performance improvements and 3 performance regressions! Performance is the same for 563 metrics, 4 unstable metrics.

scenario:iastaspectsospath-ospathbasename_aspect

  • 🟥 execution_time [+101.526µs; +108.870µs] or [+23.805%; +25.527%]

scenario:span-start

  • 🟥 execution_time [+1.474ms; +1.666ms] or [+9.535%; +10.777%]

scenario:telemetryaddmetric-1-count-metric-1-times

  • 🟥 execution_time [+312.543ns; +348.251ns] or [+14.955%; +16.663%]

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented May 7, 2026

View all feedbacks in Devflow UI.

2026-05-07 20:40:42 UTC ℹ️ Start processing command /merge


2026-05-07 20:40:52 UTC ❌ MergeQueue

This PR is rejected because it was updated

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented May 8, 2026

View all feedbacks in Devflow UI.

2026-05-08 13:22:18 UTC ℹ️ Start processing command /merge


2026-05-08 13:22:25 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals. View in MergeQueue UI.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2026-05-08 17:23:06 UTC ⚠️ MergeQueue: This merge request was unqueued

devflow unqueued this merge request: It did not become mergeable within the expected time

@datadog-datadog-prod-us1
Copy link
Copy Markdown
Contributor

datadog-datadog-prod-us1 Bot commented May 8, 2026

Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 8d1ec22 | Docs | Datadog PR Page | Give us feedback!

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented May 8, 2026

View all feedbacks in Devflow UI.

2026-05-08 17:18:35 UTC ℹ️ Start processing command /merge


2026-05-08 17:18:37 UTC ❌ MergeQueue

PR already in the queue with status waiting

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented May 8, 2026

View all feedbacks in Devflow UI.

2026-05-08 17:57:09 UTC ℹ️ Start processing command /merge


2026-05-08 17:57:14 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 53m (p90).


2026-05-08 19:57:43 UTCMergeQueue: The build pipeline has timeout

The merge request has been interrupted because the build 0 took longer than expected. The current limit for the base branch 'main' is 120 minutes.

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented May 8, 2026

View all feedbacks in Devflow UI.

2026-05-08 21:06:44 UTC ℹ️ Start processing command /merge


2026-05-08 21:06:49 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 53m (p90).


2026-05-08 23:06:41 UTC 🚨 MergeQueue: This merge request is in error

error while getting head build completion result

Details

Error: There was an error while retrieving the result for pipeline 112277682

FullStacktrace:
child workflow execution error (type: mergequeue_private.MergeQueue_WaitForChecksOrUntilIsFinal, workflowID: 019e096a-1b59-7145-9925-89cc0b1c244e_74, runID: 019e096a-45a3-7df4-abce-d3154a3604b2, initiatedEventID: 74, startedEventID: 75): child workflow execution error (type: mergequeue.MergeQueue_WaitForCompletionOfRef, workflowID: 019e096a-45a3-7df4-abce-d3154a3604b2_8, runID: 019e096a-4608-7661-9087-73144d1adad0, initiatedEventID: 8, startedEventID: 10): There was an error while retrieving the result for pipeline 112277682 (type: FlowError, retryable: false): There was an error while retrieving the result for pipeline 112277682

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented May 11, 2026

View all feedbacks in Devflow UI.

2026-05-11 17:11:30 UTC ℹ️ Start processing command /merge


2026-05-11 17:11:37 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals. View in MergeQueue UI.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2026-05-11 17:40:14 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in main is approximately 53m (p90).


2026-05-11 19:40:05 UTC 🚨 MergeQueue: This merge request is in error

error while getting head build completion result

Details

Error: There was an error while retrieving the result for pipeline 112541355

FullStacktrace:
child workflow execution error (type: mergequeue_private.MergeQueue_WaitForChecksOrUntilIsFinal, workflowID: 019e1820-0c0d-7292-b954-9d16953fcd3c_74, runID: 019e1820-3b0a-7363-94e2-1cc83db1ad39, initiatedEventID: 74, startedEventID: 75): child workflow execution error (type: mergequeue.MergeQueue_WaitForCompletionOfRef, workflowID: 019e1820-3b0a-7363-94e2-1cc83db1ad39_8, runID: 019e1820-3bae-781c-ab41-3dd3ab2ab01f, initiatedEventID: 8, startedEventID: 10): There was an error while retrieving the result for pipeline 112541355 (type: FlowError, retryable: false): There was an error while retrieving the result for pipeline 112541355

@emmettbutler
Copy link
Copy Markdown
Collaborator Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented May 12, 2026

View all feedbacks in Devflow UI.

2026-05-12 13:53:27 UTC ℹ️ Start processing command /merge


2026-05-12 13:53:35 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals. View in MergeQueue UI.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2026-05-12 14:43:07 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in main is approximately 53m (p90).


2026-05-12 16:43:48 UTCMergeQueue: The build pipeline has timeout

The merge request has been interrupted because the build 0 took longer than expected. The current limit for the base branch 'main' is 120 minutes.

@emmettbutler emmettbutler merged commit fd88e0b into main May 12, 2026
376 of 377 checks passed
@emmettbutler emmettbutler deleted the emmett.butler/name-metadata-match branch May 12, 2026 17:24
gh-worker-dd-mergequeue-cf854d Bot pushed a commit that referenced this pull request May 13, 2026
## Description

This PR: #17964 broke main because it introduced code that is incompatible with python3.9. However, microbenchmarks are running on python3.9 which broke http microbenchmarks.

This PR fixes the issue 

Co-authored-by: louis.tricot <louis.tricot@datadoghq.com>
P403n1x87 pushed a commit that referenced this pull request May 14, 2026
This change handles the fact that `__package__` and
`importlib.metadata.version(__package__)` don't always match at runtime,
specifically in the ddtrace_serverless wheel.

---------

Co-authored-by: Brett Langdon <brett.langdon@datadoghq.com>
P403n1x87 pushed a commit that referenced this pull request May 14, 2026
## Description

This PR: #17964 broke main because it introduced code that is incompatible with python3.9. However, microbenchmarks are running on python3.9 which broke http microbenchmarks.

This PR fixes the issue 

Co-authored-by: louis.tricot <louis.tricot@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants