-
Notifications
You must be signed in to change notification settings - Fork 318
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
Reduce request per iteration in IAST benchmarks #3260
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 #3260 +/- ##
=======================================
Coverage 86.03% 86.03%
=======================================
Files 194 194
Lines 7517 7517
Branches 33 33
=======================================
Hits 6467 6467
Misses 1050 1050 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
BenchmarksComparing candidate commit 6622977 in PR branch Found 44 performance improvements and 0 performance regressions! Performance is the same for 422 metrics, 26 unstable metrics. scenario:appsec-iast-no-vulnerability-control-16
scenario:appsec-iast-with-vulnerability-control-16
scenario:appsec-iast-no-vulnerability-iast-enabled-default-config-16
scenario:appsec-iast-no-vulnerability-iast-enabled-always-active-16
scenario:appsec-iast-with-vulnerability-iast-enabled-default-config-16
scenario:appsec-iast-with-vulnerability-iast-enabled-always-active-16
scenario:appsec-iast-no-vulnerability-control-18
scenario:appsec-iast-no-vulnerability-iast-enabled-default-config-18
scenario:appsec-iast-no-vulnerability-iast-enabled-always-active-18
scenario:appsec-iast-with-vulnerability-control-18
scenario:appsec-iast-with-vulnerability-iast-enabled-default-config-18
scenario:appsec-iast-with-vulnerability-iast-enabled-always-active-18
|
With 200 request by iteration, it expend 9'40", trying with 150 (it should be enough) |
* Reduce request per iteration in IAST benchmarks * Reduce a bit more * 30 iterations 200 requests
* Reduce request per iteration in IAST benchmarks * Reduce a bit more * 30 iterations 200 requests
* Reduce request per iteration in IAST benchmarks * Reduce a bit more * 30 iterations 200 requests
* Reduce request per iteration in IAST benchmarks * Reduce a bit more * 30 iterations 200 requests
* Reduce request per iteration in IAST benchmarks * Reduce a bit more * 30 iterations 200 requests
* Reduce request per iteration in IAST benchmarks * Reduce a bit more * 30 iterations 200 requests
What does this PR do?
Reduce the request made in the IAST benchmarks
Motivation
Reduce the time that test need to execute to prevent timeouts