-
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): update pytest 3.x dependencies for celery and flask_cache to 4.x #8974
chore(ci): update pytest 3.x dependencies for celery and flask_cache to 4.x #8974
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 172043 Passed, 1080 Skipped, 12h 8m 1.86s Total duration (39m 12.31s time saved) |
This unblocks #8727 . |
BenchmarksBenchmark execution time: 2024-04-15 11:33:01 Comparing candidate commit 55aa862 in PR branch Found 7 performance improvements and 3 performance regressions! Performance is the same for 191 metrics, 9 unstable metrics. scenario:flasksimple-appsec-telemetry
scenario:httppropagationextract-datadog_tracecontext_tracestate_not_propagated_on_trace_id_no_match
scenario:httppropagationextract-wsgi_invalid_trace_id_header
scenario:httppropagationinject-ids_only
scenario:httppropagationinject-with_dd_origin
scenario:httppropagationinject-with_sampling_priority
scenario:httppropagationinject-with_tags
scenario:httppropagationinject-with_tags_invalid
scenario:httppropagationinject-with_tags_max_size
scenario:span-start-finish
|
This bumps pytest dependencies pinned on 3.10.1 (which is from late 2018) to
~=4.0
which should install 4.6.11 which dates back to mid-2020.It also bumps
pytest-cov
to~=3.0
due to some functions having been removed in2.x
that we need for thecoverage
integration.I'm using
~=
for specifying versions even though the versions being used are old enough that they're no longer getting updates, so this effectively a version pin. I don't think that really matters, though, and I think in general ourriotfile.py
probably benefits from using compatible release clauses.Checklist
changelog/no-changelog
is set@DataDog/apm-tees
.Reviewer Checklist