Skip to content

xDS interop GCE framework: pin grpcio-tools to use protobuf 3.x#31191

Merged
sergiitk merged 2 commits intogrpc:masterfrom
sergiitk:xds-interop-gce-protobuf
Sep 30, 2022
Merged

xDS interop GCE framework: pin grpcio-tools to use protobuf 3.x#31191
sergiitk merged 2 commits intogrpc:masterfrom
sergiitk:xds-interop-gce-protobuf

Conversation

@sergiitk
Copy link
Copy Markdown
Member

@sergiitk sergiitk commented Sep 30, 2022

@sergiitk sergiitk marked this pull request as ready for review September 30, 2022 21:19
@sergiitk sergiitk added release notes: no Indicates if PR should not be in release notes area/psm interop labels Sep 30, 2022
@sergiitk sergiitk requested a review from gnossen September 30, 2022 21:20
Copy link
Copy Markdown
Contributor

@gnossen gnossen left a comment

Choose a reason for hiding this comment

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

Can you please also add a TODO to unpin? This is good signal for us currently when this breaks.

@sergiitk sergiitk enabled auto-merge (squash) September 30, 2022 23:18
@sergiitk sergiitk merged commit 203bc0d into grpc:master Sep 30, 2022
@sergiitk sergiitk deleted the xds-interop-gce-protobuf branch October 1, 2022 00:23
sergiitk added a commit to sergiitk/grpc that referenced this pull request Oct 1, 2022
sergiitk added a commit to sergiitk/grpc that referenced this pull request Oct 1, 2022
sergiitk added a commit to sergiitk/grpc that referenced this pull request Oct 1, 2022
sergiitk added a commit to sergiitk/grpc that referenced this pull request Oct 1, 2022
sergiitk added a commit to sergiitk/grpc that referenced this pull request Oct 1, 2022
sergiitk added a commit to sergiitk/grpc that referenced this pull request Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/psm interop bloat/none imported Specifies if the PR has been imported to the internal repository per-call-memory/neutral per-channel-memory/neutral release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants