-
Notifications
You must be signed in to change notification settings - Fork 412
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
chore(ci): convert flask_sqli benchmark to use Flask test client instead of gunicorn #9253
Conversation
Datadog ReportBranch report: ❌ 162 Failed (0 Known Flaky), 157851 Passed, 3048 Skipped, 3h 53m 55.32s Total duration (18m 13.69s time saved) ❌ Failed Tests (162)
New Flaky Tests (217)
|
BenchmarksBenchmark execution time: 2024-05-14 12:37:24 Comparing candidate commit 5bff0c8 in PR branch Found 8 performance improvements and 0 performance regressions! Performance is the same for 167 metrics, 9 unstable metrics. scenario:flasksqli-appsec-enabled
scenario:flasksqli-baseline
scenario:flasksqli-iast-enabled
scenario:flasksqli-tracer-enabled
|
Follow up from #8902 to also convert the
flask_sqli
benchmark to using the Flask test client instead of gunicorn.This cuts the benchmark runtime from 25min to 5min.
We should be testing the same code paths, but we avoid the need to spin up a subprocess/server and make network requests to it.
We also refactored some common bits from both
flask_simple
andflask_sqli
to ensure we are configuring them the same way.This will look like a performance improvement, but it isn't. It is the test itself getting faster.
Checklist
changelog/no-changelog
is set@DataDog/apm-tees
.Reviewer Checklist