-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Fix tracemalloc usage error #20355
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
Fix tracemalloc usage error #20355
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a request to take this opportunity to improve an inline comment.
any chance to add some tests for this? 🙏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving, I'll follow-up with that inline comment to understand it better and maybe even remove it.
Thanks a ton for the tests!
* Fix tracemalloc usage error * Add changelog * Add tests * Run formatter 87a21c0
What does this PR do?
Fixes a bug introduced in #19197 where using
tracemalloc_debug: true
in thedatadog.yaml
causes integrations to error out.Motivation
https://datadoghq.atlassian.net/browse/AGENT-13797
Review checklist (to be filled by reviewers)
qa/skip-qa
label if the PR doesn't need to be tested during QA.backport/<branch-name>
label to the PR and it will automatically open a backport PR once this one is merged