Skip to content

Add upper limit on grpcio-tools version#671

Closed
alexnikulkov wants to merge 1 commit intofacebookresearch:mainfrom
alexnikulkov:export-D38870120
Closed

Add upper limit on grpcio-tools version#671
alexnikulkov wants to merge 1 commit intofacebookresearch:mainfrom
alexnikulkov:export-D38870120

Conversation

@alexnikulkov
Copy link
Contributor

Summary:
CircleCI tests started failing. Example: https://app.circleci.com/pipelines/github/facebookresearch/ReAgent/2490/workflows/efbceb68-9a01-4889-8d82-3d4167af4643/jobs/24780
Root cause is a significant jump in protobuf version from 3.19.4 to 4.21.5. This was caused by an an updated dependency on protobuf in grcpio-tools.
I'm adding an upper limit on grcpio-tools version to prevent this error

Differential Revision: D38870120

Summary:
CircleCI tests started failing. Example: https://app.circleci.com/pipelines/github/facebookresearch/ReAgent/2490/workflows/efbceb68-9a01-4889-8d82-3d4167af4643/jobs/24780
Root cause is a significant jump in protobuf version from 3.19.4 to 4.21.5. This was caused by an an updated dependency on protobuf in grcpio-tools.
I'm adding an upper limit on grcpio-tools version to prevent this error

Differential Revision: D38870120

fbshipit-source-id: 37ef2fe3289e0f58791e1d6f5d34d390737fe4fb
@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D38870120

xuruiyang pushed a commit that referenced this pull request Sep 20, 2025
Summary:
Pull Request resolved: #671

CircleCI tests started failing. Example: https://app.circleci.com/pipelines/github/facebookresearch/ReAgent/2490/workflows/efbceb68-9a01-4889-8d82-3d4167af4643/jobs/24780
Root cause is a significant jump in protobuf version from 3.19.4 to 4.21.5. This was caused by an an updated dependency on protobuf in grcpio-tools.
I'm adding an upper limit on grcpio-tools version to prevent this error

Reviewed By: BerenLuthien, czxttkl

Differential Revision: D38870120

fbshipit-source-id: 10d2e619aa10c2b03272b8305681792feefd06f4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants