Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace libddprof dependency with libdatadog, and upgrade to 0.7.0 #2061

Merged
merged 5 commits into from
Jul 11, 2022

Conversation

ivoanjo
Copy link
Member

@ivoanjo ivoanjo commented May 31, 2022

The libddprof dependency has been renamed libdatadog.

This PR updates dd-trace-rb to use the new version, including a few API changes from the 0.6 to 0.7 release.

I'm marking this PR as a draft because the only libdatadog version right now is 0.7.0.1.0.rc1 and I want to avoid releasing a rc1 to customers. Once the final 0.7.0 release hits, we just need to change the gemspec and merge.

`libdatadog` is the new name of `libddprof`, and thus the gem name
changed to match.

This commit includes only the Ruby-level changes; C API changes from
the 0.6.0 to 0.7.0 release will be fixed separately.
Now that we can explicitly get rid of the request, we can simplify the
`pending_exception` code path when reporting a profile.
@ivoanjo ivoanjo force-pushed the ivoanjo/upgrade-to-libdatadog-0.7.0 branch from fcb20ff to 142d404 Compare July 8, 2022 10:24
@ivoanjo
Copy link
Member Author

ivoanjo commented Jul 8, 2022

Update: I've just rebased this PR on top of current master, as well as doing one more commit to clean up more references to libddprof (a bunch of apis are still prefixed with ddprof_ffi, but nothing we can do from the dd-trace-rb side).

✅. Libdatadog 0.7.0 should be out on rubygems.org today (just need an approval on the release PR) and then this PR will be in good merging shape :) (Done)

I plan to keep it marked as draft until we put out dd-trace-rb 1.2.0 (no need to risk last-minute stability), but otherwise it's ready.

@ivoanjo ivoanjo marked this pull request as ready for review July 11, 2022 15:06
@ivoanjo ivoanjo requested a review from a team July 11, 2022 15:06
@ivoanjo
Copy link
Member Author

ivoanjo commented Jul 11, 2022

dd-trace-rb 1.2.0 is now out, so I'm pressing the magic button!

@ivoanjo ivoanjo merged commit dc534a1 into master Jul 11, 2022
@ivoanjo ivoanjo deleted the ivoanjo/upgrade-to-libdatadog-0.7.0 branch July 11, 2022 15:07
@github-actions github-actions bot added this to the 1.3.0 milestone Jul 11, 2022
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.

None yet

2 participants