Skip to content

[PROF-11680] Add support for profiling Ruby 3.5.0-preview1#4600

Merged
ivoanjo merged 3 commits into
masterfrom
ivoanjo/prof-11680-ruby-3_5-preview1
Apr 24, 2025
Merged

[PROF-11680] Add support for profiling Ruby 3.5.0-preview1#4600
ivoanjo merged 3 commits into
masterfrom
ivoanjo/prof-11680-ruby-3_5-preview1

Conversation

@ivoanjo

@ivoanjo ivoanjo commented Apr 24, 2025

Copy link
Copy Markdown
Member

What does this PR do?

This PR adds support for profiling Ruby 3.5.0-preview1. The only work needed was importing the latest headers in
DataDog/datadog-ruby_core_source#15 so this PR adds the new 3.4.1 release of the headers as the minimum version allowed.

Motivation:

By setting the minimum version of datadog-ruby_core_source to the latest release, we make sure bundler picks it up, and doesn't try to use an incorrect release with Ruby 3.5.

Change log entry

Yes. Add support for profiling Ruby 3.5.0-preview1

Additional Notes:

N/A

How to test the change?

We don't yet test with 3.5 on CI; I tested locally that the profiler test suite is working fine with 3.4 and 3.5.

**What does this PR do?**

This PR adds support for profiling Ruby 3.5.0-preview1. The only
work needed was importing the latest headers in
DataDog/datadog-ruby_core_source#15
so this PR adds the new 3.4.1 release of the headers as the minimum
version allowed.

**Motivation:**

By setting the minimum version of `datadog-ruby_core_source` to the
latest release, we make sure bundler picks it up, and doesn't try
to use an incorrect release with Ruby 3.5.

**Additional Notes:**

N/A

**How to test the change?**

We don't yet test with 3.5 on CI; I tested locally that the profiler
test suite is working fine with 3.4 and 3.5.
@ivoanjo
ivoanjo requested a review from a team as a code owner April 24, 2025 09:26
@ivoanjo ivoanjo changed the title [PROF-11680] Add support for profiling 3.5.0-preview1 [PROF-11680] Add support for profiling Ruby 3.5.0-preview1 Apr 24, 2025
@pr-commenter

pr-commenter Bot commented Apr 24, 2025

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2025-04-24 14:43:54

Comparing candidate commit 69164d4 in PR branch ivoanjo/prof-11680-ruby-3_5-preview1 with baseline commit b3f7d09 in branch master.

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

@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Apr 24, 2025

Copy link
Copy Markdown
Contributor

Datadog Report

Branch report: ivoanjo/prof-11680-ruby-3_5-preview1
Commit report: 69164d4
Test service: dd-trace-rb

✅ 0 Failed, 20837 Passed, 1373 Skipped, 3m 42.34s Total Time

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.73%. Comparing base (b3f7d09) to head (69164d4).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4600   +/-   ##
=======================================
  Coverage   97.73%   97.73%           
=======================================
  Files        1411     1411           
  Lines       86225    86225           
  Branches     4354     4354           
=======================================
+ Hits        84269    84274    +5     
+ Misses       1956     1951    -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ivoanjo
ivoanjo merged commit 587bf0b into master Apr 24, 2025
@ivoanjo
ivoanjo deleted the ivoanjo/prof-11680-ruby-3_5-preview1 branch April 24, 2025 14:52
@github-actions github-actions Bot added this to the 2.16.0 milestone Apr 24, 2025
@Strech Strech mentioned this pull request May 19, 2025
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.

4 participants