Skip to content

dependabot: Add a migration script for grpc updates#471

Merged
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:fix-dependabot-grpc
Apr 24, 2026
Merged

dependabot: Add a migration script for grpc updates#471
llucax merged 1 commit intofrequenz-floss:v0.x.xfrom
llucax:fix-dependabot-grpc

Conversation

@llucax
Copy link
Copy Markdown
Contributor

@llucax llucax commented Apr 24, 2026

Dependabot fails to update grpc/protobuf dependencies correctly, as it limits itself to update build dependencies, but leave the runtime dependencies untouched.

This doesn't work because protobuf/grpc need to be generated with a version that is equals or older to the runtime version, so if we only bump the build dependency, a project might still depend on an older version at runtime, and we should not allow that.

This PR adds an auto-migration workflow to fix this, while also ensuring the major version for protobuf is bumped accordingly when a major version bump occurs.

This is an experimental change. If this work the upgrade script will be moved to a separate repository and the workflow added to repo-config.

@llucax llucax requested a review from a team as a code owner April 24, 2026 11:11
@llucax llucax requested review from cyiallou and removed request for a team April 24, 2026 11:11
@github-actions github-actions Bot added the part:tooling Affects the development tooling (CI, deployment, dependency management, etc.) label Apr 24, 2026
@llucax llucax marked this pull request as draft April 24, 2026 11:13
@llucax
Copy link
Copy Markdown
Contributor Author

llucax commented Apr 24, 2026

Marking as draft because I will try to do some tests in my fork first.

Dependabot fails to update grpc/protobuf dependencies correctly, as it
limits itself to update build dependencies, but leave the runtime
dependencies untouched.

This doesn't work because protobuf/grpc need to be generated with a
version that is equals or older to the runtime version, so if we only
bump the build dependency, a project might still depend on an older
version at runtime, and we should not allow that.

This commit adds an auto-migration workflow to fix this, while also
ensuring the major version for protobuf is bumped accordingly when a
major version bump occurs.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax force-pushed the fix-dependabot-grpc branch from cc5fa62 to 10cf124 Compare April 24, 2026 12:40
@llucax llucax marked this pull request as ready for review April 24, 2026 12:40
Copy link
Copy Markdown
Contributor

@tiyash-basu-frequenz tiyash-basu-frequenz left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@llucax llucax added this pull request to the merge queue Apr 24, 2026
Merged via the queue into frequenz-floss:v0.x.x with commit 26c6fca Apr 24, 2026
9 checks passed
@llucax llucax deleted the fix-dependabot-grpc branch April 24, 2026 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