Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

• User @TeBoring isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

It is useful to provide link to the last update #19182 - most changes should be of the same nature.

@TeBoring for some reason the CLA check is failing - can you doublecheck please?

CLA: https://identity.linuxfoundation.org/projects/cncf you should be able to sign as google employee.

The c# error is something we need to take a look into. It might also be that 3.9.0 might contain a breaking change that needs to be rolled back.

  Grpc.Microbenchmarks -> /var/local/git/grpc/src/csharp/Grpc.Microbenchmarks/bin/Debug/net461/Grpc.Microbenchmarks.exe
Math.cs(39,17): error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments [/var/local/git/grpc/src/csharp/Grpc.Examples/Grpc.Examples.csproj]
Math.cs(40,17): error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments [/var/local/git/grpc/src/csharp/Grpc.Examples/Grpc.Examples.csproj]
Math.cs(41,17): error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments [/var/local/git/grpc/src/csharp/Grpc.Examples/Grpc.Examples.csproj]
Math.cs(42,17): error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments [/var/local/git/grpc/src/csharp/Grpc.Examples/Grpc.Examples.csproj]
Math.cs(43,17): error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments [/var/local/git/grpc/src/csharp/Grpc.Examples/Grpc.Examples.csproj]

Other problems:
https://source.cloud.google.com/results/invocations/c03ae785-c291-4a23-95a9-c0b8a91decf6/targets/github%2Fgrpc%2Fsanity_linux_dbg_native/tests

The C# problem likely has to do with this change:
protocolbuffers/protobuf@9e89b6e#diff-a11488f894c3a2f0a362c3ea33941a88R107

CC @ObsidianMinor (probably related to protocolbuffers/protobuf#5350)

I signed

3.9.0 was delisted until a 3.9.1 release can be made to with these changes. The C# issues are probably related to that.

Looks like the test is using new protoc but old c# runtime? @jtattermusch is the failure in this case, and why old c# runtime is used in Grpc's test?

Protobuf only gurantee old generated code work with new runtime. Not gurantee new generated code work with old runtime

Ok, I think I now understand what happened:

• this PR attempts to increase C# protobuf runtime version in Version.csproj.include, but it turns out that's a dead copy that doesn't influence anything, so the protobuf runtime is not really updated, but the code is regenerated. That explains the C# build error.

I created #19679 to fix the problem. After it's merged, this PR needs to be updated to modify src/csharp/build/dependencies.props instead of Version.csproj.include

Superseded by #19867