⬆️ 🔒️🔨 Maintenance: repo-wide test/tools upgrades and new batch-upgrades scripts for otel and pydantic
#7973
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7973 +/- ##
==========================================
- Coverage 87.83% 86.76% -1.07%
==========================================
Files 1805 1853 +48
Lines 69606 71485 +1879
Branches 1254 1254
==========================================
+ Hits 61135 62026 +891
- Misses 8106 9097 +991
+ Partials 365 362 -3
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
pcrespov
force-pushed
the
mai/upgrade-requirements-tests
branch
from
b4a823e to
f1ba7c6
Compare
- Updated `anyio` from 4.8.0 to 4.9.0 - Updated `certifi` from 2025.1.31 to 2025.6.15 - Updated `click` from 8.1.8 to 8.2.1 - Updated `h11` from 0.14.0 to 0.16.0 - Updated `httpcore` from 1.0.7 to 1.0.9 - Updated `orjson` from 3.10.15 to 3.10.18 - Updated `packaging` from 24.2 to 25.0 - Updated `pydantic` from 2.10.6 to 2.11.7 - Updated `pydantic-core` from 2.27.2 to 2.33.2 - Updated `pydantic-extra-types` from 2.10.2 to 2.10.5 - Updated `pygments` from 2.19.1 to 2.19.2 - Updated `python-dotenv` from 1.0.1 to 1.1.1 - Updated `rich` from 13.9.4 to 14.0.0 - Updated `tenacity` from 9.0.0 to 9.1.2 - Updated `typer` from 0.15.2 to 0.16.0 - Updated `typing-extensions` from 4.12.2 to 4.14.0 - Updated `urllib3` from 2.3.0 to 2.5.0 - Updated test dependencies similarly across `_test.txt`, `_tools.txt`, and `swarm-deploy` requirements.
… 1.34.1 and instrumentation packages to version 0.55b1. Remove deprecated package references from requirements files.
pcrespov
changed the title
pcrespov
changed the title
otel and pydantic
pcrespov
force-pushed
the
mai/upgrade-requirements-tests
branch
from
f1ba7c6 to
2c9f9df
Compare
pcrespov
requested review from
GitHK,
giancarloromeo,
matusdrobuliak66 and
sanderegg
as code owners
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades various test and tooling dependencies across the repo and adds scripts/recipes for batch upgrading the OpenTelemetry and Pydantic families.
- Repo-wide bump of test and tooling requirements
- Repo-wide upgrade of Pydantic family (2.10→2.11.x) and code adaptations
- Repo-wide upgrade of OpenTelemetry family (0.51/1.30→0.55/1.34) and new batch-upgrade recipes
Reviewed Changes
Copilot reviewed 123 out of 123 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/swarm-deploy/requirements/_tools.txt | Updated versions of dev tools (black, mypy, pylint, etc.) |
| tests/swarm-deploy/requirements/_test.txt | Updated test dependencies; added constraint comments |
| tests/public-api/requirements/_tools.txt | Updated versions of dev tools |
| tests/public-api/requirements/_test.txt | Updated test dependencies |
| tests/public-api/requirements/_base.txt | Updated base dependencies |
| tests/performance/requirements/_tools.txt | Updated dev tool versions |
| tests/performance/requirements/_test.txt | Updated test deps for performance tests |
| tests/environment-setup/requirements/requirements.txt | Updated pytest and pydantic versions |
| tests/e2e/requirements/requirements.txt | Updated e2e test deps |
| tests/e2e-playwright/requirements/_tools.txt | Updated dev tools for Playwright tests |
| tests/e2e-playwright/requirements/_test_wo_playwright.txt | Updated test deps |
| tests/e2e-playwright/requirements/_test.txt | Updated Playwright test deps |
| services/web/server/tests/unit/isolated/test__configs.py | Added note on protobuf compatibility issue |
| services/web/server/requirements/_tools.txt | Bumped setuptools |
| services/web/server/requirements/_test.txt | Upgraded h11/httpcore, pydantic |
| services/web/server/requirements/_base.txt | Upgraded otel/pydantic and collapsed deprecated entries |
| services/storage/requirements/_tools.txt | Bumped setuptools |
| services/storage/requirements/_test.txt | Upgraded h11/httpcore, pydantic |
| services/storage/requirements/_base.txt | Bumped protos, otel entries |
| services/resource-usage-tracker/requirements/_tools.txt | Bumped setuptools |
| services/resource-usage-tracker/requirements/_test.txt | Upgraded h11/httpcore, pydantic |
| services/resource-usage-tracker/requirements/_base.txt | Upgraded googleapis, h11, httpcore |
| services/payments/requirements/_tools.txt | Bumped setuptools |
| services/payments/requirements/_test.txt | Upgraded h11/httpcore |
| services/payments/requirements/_base.txt | Removed deprecated entries; bumped otel |
| services/notifications/requirements/_tools.txt | Bumped setuptools |
| services/notifications/requirements/_test.txt | Upgraded h11/httpcore |
| services/notifications/requirements/_base.txt | Removed deprecated entries; bumped otel |
| services/migration/requirements/_tools.txt | Bumped dev tools |
| services/migration/requirements/_test.txt | Upgraded certifi, pytest, pydantic |
| services/migration/requirements/_base.txt | Removed deprecated entries; bumped otel |
| services/efs-guardian/requirements/_tools.txt | Bumped setuptools |
| services/efs-guardian/requirements/_test.txt | Upgraded h11/httpcore, pydantic |
| services/efs-guardian/requirements/_base.txt | Bumped setuptools |
Comments suppressed due to low confidence (2)
tests/swarm-deploy/requirements/_test.txt:167
- The long, repetitive constraint comment blocks make the file hard to maintain; consider consolidating constraint references into a shared include or using a shorter summary.
# -c requirements/../../../packages/service-library/requirements/../../../packages/common-library/requirements/../../../requirements/constraints.txt
services/web/server/requirements/_base.txt:255
- This repeated list of constraint comments is verbose and difficult to update; consider grouping common constraints or referencing a single shared constraints file to reduce duplication.
# -c requirements/../../../../packages/common-library/requirements/../../../requirements/constraints.txt
matusdrobuliak66
approved these changes
Jun 27, 2025
bisgaard-itis
approved these changes
Jun 27, 2025
giancarloromeo
approved these changes
Jun 27, 2025
…tings model configuration
|
Member
Author
|
not sustainable. will split in smaller PRs |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What do these changes do?
⬆️ Repo-wide upgrade of requirements for testing and tooling
make reqs⬆️ 🔒️ Security Updates
protobuf>=5.29.5 : GHSA-8qvm-5x2c-j2w7Could not upgrade for now🔨 New recipes to batch-updates for
otelandpydanticfamilies⬆️ Upgraded
pydantic-family repo-widepydantic.warnings.PydanticDeprecatedSince211: Accessing the 'model_fields' attribute on the instance is deprecated. Instead, you should access this attribute from the model class. Deprecated in Pydantic V2.11 to be removed in V3.0.populate_by_namebyvalidate_by_name. SEEpackages/models-library/tests/test__pydantic_models.pyextra=forbid!!⬆️ Upgraded
otel-family repo-wide