Upgrade gRPC version

[markmandel]

We should upgrade gRPC. k8s.io/apiserver would like it to be 1.23.0 (even though we don't use that part of the code), so probably makes sense to align it with what libraries we use are requesting.

(Unless anyone can think of a good reason to go straight to latest?)

https://github.com/googleforgames/agones/blob/master/build/README.md#make-gen-sdk-grpc will be required here to update gRPC based SDKs and make sure they work. Might be worth splitting that work up as well as separate PRs.

Context: #1794 (comment)

[markmandel]

One other thought on this - if we want to upgrade grpc-gateway (maybe to v2?) we should upgrade gRPC to the appropriate version first.

[markmandel]

Reopening this, to cover gRPC SDK tooling.

We did update gRPC for the SDK server, but haven't yet done any of the work to upgrade the codegen for the client sdks and related. They may not change, but usually there is some change in the generated code from gRPC between versions. At the very least we should check.

• update the SDK base image grpc version and rebuild the image. (this can take a while, have had to do it manually and push it to gcr because cloud build doesn't like how long it takes).
• regen allocated API endpoints: make gen-allocation-grpc (looks like this one isn't documented)
• regen all client sdks: make gen-sdk-grpc
• update the version number in C++ Cmake scripts
• Probably worth doing a global find for "1.20.1" to see if there is anything else that needs updating.
• Fix all the things until they pass 😄

[dbotha]

@markmandel This upgrade would be very welcome 😅 Currently I have to apply a minor patch to gRPC to get the c++ SDK to compile under Alpine Linux: d93e057

This wouldn't be required under more recent versions of gRPC (v1.24.0 onwards). See resolved issue grpc/grpc#19964

Kind regards,

Deon

[markmandel]

Noting - this is the current gRPC version we are using server side, so we need to bring the client side SDK tooling inline with this version:
https://github.com/googleforgames/agones/blob/main/go.mod#L33

google.golang.org/grpc v1.27.1

[markmandel]

Just noting that there are still a few more checkboxes in the list that need completing before we can close this out.
#1797 (comment)

We might be able to close off "Probably worth doing a global find for "1.20.1" to see if there is anything else that needs updating." - @SaitejaTamma did you end up doing a search? If so, we can tick this item off.

[SaitejaTamma]

Yes, we can tick this off

[roberthbailey]

@markmandel - with the recent gRPC updates, can we close this tracking ticket?

[markmandel]

Yes! Let's do it!