Skip to content

Bump protobuf-related dependencies#312

Merged
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:protobuf6
Jun 10, 2025
Merged

Bump protobuf-related dependencies#312
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:protobuf6

Conversation

@llucax
Copy link
Copy Markdown
Contributor

@llucax llucax commented Jun 10, 2025

Also ignore warnings about protobuf gencode version, as it is guaranteed by protobuf that 2 consecutive major versions are compatible. We are aware that after that we need to update the depencencies, and we'll get an error if we forget anyways.

Also ignore warnings about protobuf gencode version, as it is guaranteed
by protobuf that 2 consecutive major versions are compatible. We are
aware that after that we need to update the depencencies, and we'll get
an error if we forget anyways.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
Copilot AI review requested due to automatic review settings June 10, 2025 12:49
@llucax llucax requested review from a team as code owners June 10, 2025 12:49
@github-actions github-actions Bot added the part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) label Jun 10, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Bumps protobuf and gRPC dependencies to newer versions and adjusts pytest warning handling.

  • Update pinned versions for protobuf, grpcio-tools, and grpcio
  • Widen dependency ranges to allow the new major versions
  • Simplify pytest addopts and add filterwarnings to manage deprecation and specific UserWarnings
Comments suppressed due to low confidence (1)

pyproject.toml:40

  • The upper bound < 8 exceeds the documented two-major-version compatibility window for protobuf. Consider restricting it to < 7 to ensure runtime compatibility.
"protobuf >= 6.31.1, < 8", # Do not widen beyond 8!

Comment thread pyproject.toml
@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Jun 10, 2025

Routine update applied to many repos, so also force-merging after tests are done.

@llucax llucax merged commit 5777aae into frequenz-floss:v0.x.x Jun 10, 2025
7 checks passed
@llucax llucax deleted the protobuf6 branch June 10, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:tooling Affects the development tooling (CI, deployment, dependency management, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants