Skip to content

chore(crashtracker): local testing and profiling frame on 3.15 - #17851

Closed
gyuheon0h wants to merge 1 commit into
vlad/ddtracepy-315-tracing-wrappingfrom
gyuheon0h/PROF-14459-3_15
Closed

chore(crashtracker): local testing and profiling frame on 3.15#17851
gyuheon0h wants to merge 1 commit into
vlad/ddtracepy-315-tracing-wrappingfrom
gyuheon0h/PROF-14459-3_15

Conversation

@gyuheon0h

Copy link
Copy Markdown
Contributor

Description

Testing

Risks

Additional Notes

gyuheon0h commented May 1, 2026

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@gyuheon0h gyuheon0h changed the title Verify CT on 3.15 chore(crashtracker): crashtracker on 3.15 May 1, 2026
@cit-pr-commenter-54b7da

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

Copy link
Copy Markdown

Codeowners resolved as

ddtrace/internal/datadog/profiling/dd_wrapper/src/native_call_tracker.cpp  @DataDog/profiling-python
ddtrace/internal/datadog/profiling/dd_wrapper/src/sample.cpp            @DataDog/profiling-python
ddtrace/internal/datadog/profiling/stack/src/echion/frame.cc            @DataDog/profiling-python
riotfile.py                                                             @DataDog/apm-python

@pr-commenter

pr-commenter Bot commented May 1, 2026

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2026-05-01 20:48:14

Comparing candidate commit 106a8e5 in PR branch gyuheon0h/PROF-14459-3_15 with baseline commit d52c831 in branch vlad/ddtracepy-315-tracing-wrapping.

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

scenario:iastaspects-lstrip_noaspect

  • 🟥 execution_time [+15.864µs; +24.843µs] or [+9.009%; +14.109%]

scenario:iastaspects-stringio_noaspect

  • 🟥 execution_time [+25.397µs; +31.047µs] or [+7.257%; +8.872%]

scenario:iastaspectsospath-ospathbasename_aspect

  • 🟥 execution_time [+86.838µs; +97.239µs] or [+20.490%; +22.945%]

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

  • 🟥 execution_time [+170.389ns; +201.715ns] or [+8.071%; +9.554%]

@gyuheon0h
gyuheon0h changed the base branch from main to graphite-base/17851 May 1, 2026 20:03
@gyuheon0h
gyuheon0h force-pushed the gyuheon0h/PROF-14459-3_15 branch from 5b4280f to 9025766 Compare May 1, 2026 20:03
@gyuheon0h
gyuheon0h changed the base branch from graphite-base/17851 to vlad/ddtracepy-315-tracing-wrapping May 1, 2026 20:04
@gyuheon0h gyuheon0h added the changelog/no-changelog A changelog entry is not required for this PR. label May 1, 2026
@gyuheon0h
gyuheon0h force-pushed the gyuheon0h/PROF-14459-3_15 branch from 9025766 to 106a8e5 Compare May 1, 2026 20:10
@gyuheon0h gyuheon0h changed the title chore(crashtracker): crashtracker on 3.15 chore(crashtracker): local testing and profiling frame on 3.15 May 1, 2026
@datadog-datadog-prod-us1

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

Copy link
Copy Markdown
Contributor

Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

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

@github-actions github-actions Bot added the stale label Jun 1, 2026
@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically closed after a period of inactivity.
After this much time, it will likely be easier to open a new pull request with the
same changes than to update this one from the base branch. Please comment or reopen
if you think this pull request was closed in error.

@github-actions github-actions Bot closed this Jun 2, 2026
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. stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant