-
Notifications
You must be signed in to change notification settings - Fork 306
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 IAST benchmarks #3193
Add IAST benchmarks #3193
Conversation
Overall package sizeSelf size: 4.36 MB Dependency sizes
🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report
@@ Coverage Diff @@
## master #3193 +/- ##
=======================================
Coverage 86.02% 86.02%
=======================================
Files 194 194
Lines 7514 7514
Branches 33 33
=======================================
Hits 6464 6464
Misses 1050 1050 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
BenchmarksComparing candidate commit a714dde in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 406 metrics, 25 unstable metrics. scenario:log-without-log-18
|
8801411
to
b2b4810
Compare
7d09eac
to
61b3819
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.
Make sure env vars are applied correctly and test the thing altogether, then LGTM
2f7a5b0
to
cdc9c4b
Compare
0e4526c
to
4cd3313
Compare
I put the PR back in draft if that's ok with you |
3a13966
to
aa620bf
Compare
db4404c
to
0301193
Compare
0301193
to
7566bc2
Compare
Co-authored-by: simon-id <simon.id@datadoghq.com>
* appsec-iast benchmark tests * writing fixes Co-authored-by: simon-id <simon.id@datadoghq.com> * small fixes --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
* appsec-iast benchmark tests * writing fixes Co-authored-by: simon-id <simon.id@datadoghq.com> * small fixes --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
* appsec-iast benchmark tests * writing fixes Co-authored-by: simon-id <simon.id@datadoghq.com> * small fixes --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
* appsec-iast benchmark tests * writing fixes Co-authored-by: simon-id <simon.id@datadoghq.com> * small fixes --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
* appsec-iast benchmark tests * writing fixes Co-authored-by: simon-id <simon.id@datadoghq.com> * small fixes --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
* appsec-iast benchmark tests * writing fixes Co-authored-by: simon-id <simon.id@datadoghq.com> * small fixes --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
What does this PR do?
Executes and compares with master benchmark tests with DD_IAST_ENABLED.
Motivation
Make sure that we are not adding regressions in the PRs