-
Notifications
You must be signed in to change notification settings - Fork 419
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 profiler runs on regression #1798
Conversation
433d4a5
to
898b449
Compare
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.
LGTM, but the bash is getting a bit wild in terms of duplications. Extracting some stuff into a simple bash functions might help a lot with keeping the code readable and maintainable.
Not a blocker for merging this, but if you have some time ...
435334c
to
8f52069
Compare
8f52069
to
d4fa189
Compare
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.
LGTM. 🚀
* Add profiler runs on regression * Add uploading to DD backend * Remove duplication of code in bash scripts --------- Co-authored-by: Felix Geisendörfer <felix@datadoghq.com>
* Add profiler runs on regression * Add uploading to DD backend * Remove duplication of code in bash scripts --------- Co-authored-by: Felix Geisendörfer <felix@datadoghq.com>
What does this PR do?
This PR adds profiler runs for benchmarks if regression was detected. This should add more confidence in results and allow to see what caused change in performance.
Motivation
Describe how to test/QA your changes
Reviewer's Checklist
Triage
milestone is set.