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

Add configuration to save profiles to disk #1837

Merged
merged 3 commits into from
Dec 20, 2022
Merged

Conversation

morrisonlevi
Copy link
Collaborator

@morrisonlevi morrisonlevi commented Dec 19, 2022

Description

Being able to save the profiles to disk instead of sending them to Datadog is nicer for certain kinds of tests, including the prof-correctness ones that Erwan and others are working on.

Readiness checklist

  • Changelog has been added to the release document.
  • Tests added for this feature/bug.

Reviewer checklist

  • Appropriate labels assigned.
  • Milestone is set.

@morrisonlevi morrisonlevi added dev/testing profiling Relates to the Continuous Profiler labels Dec 19, 2022
@morrisonlevi morrisonlevi added this to the 0.83.0 milestone Dec 19, 2022
@morrisonlevi morrisonlevi marked this pull request as ready for review December 19, 2022 21:09
@morrisonlevi morrisonlevi requested a review from a team as a code owner December 19, 2022 21:09
Copy link
Member

@realFlowControl realFlowControl left a comment

Choose a reason for hiding this comment

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

Looks good to me 🎉

@morrisonlevi morrisonlevi merged commit 319b2af into master Dec 20, 2022
@morrisonlevi morrisonlevi deleted the levi/pprof-disk branch December 20, 2022 22:42
@r1viollet
Copy link
Collaborator

Nice 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/testing profiling Relates to the Continuous Profiler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants