jaegertracing / docker-protobuf

An all-inclusive protoc Docker image for the Jaeger project
https://hub.docker.com/r/jaegertracing/protobuf
Apache License 2.0
93 stars 35 forks source link

Update dependencies #29

Closed mmorel-35 closed 8 months ago

mmorel-35 commented 11 months ago

Summary

Update Alpine to 3.18 Update Golang to 1.21.3 Update grpc to 1.50.0 Update grpc-gateway to 2.18.0 Add protoc-gen-go-grpc 1.3.0 Update actions/checkout to v4 Update upx to 4.2.1 Update protoc-gen-lint to 0.3.0

Changes

mmorel-35 commented 11 months ago

Hi @yurishkuro , This is compiling with grpc 1.50. I tried 1.59 and 1.55 which are both failing to compile after 1h11. Can we keep this upgrades for later and get a release with this version ?

yurishkuro commented 11 months ago

I think we need to add the jaeger-idl repo (or even main jaeger) as a submodule and run a build there as part of the CI in this repo. Otherwise it's very hard to verify that the changes are not breaking something when compiling the IDL.