-
Notifications
You must be signed in to change notification settings - Fork 10.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build_docker_go failure #9011
Comments
I think I fixed this in #8845 Should we backport that change to the go docker file? |
To |
Werp. Seen again against master: https://grpc-testing.appspot.com/job/gRPC_interop_pull_requests/10078/console |
That PR is 1500 commits behind master and is missing my fix in #8845. I will update the thread accordingly |
No recent activity, closing. |
@menghanl I remember you had a PR last week that is supposed to fix this? |
Still seeing this in PR #10349 |
@ncteisen, just saw this in a new PR against v1.2.x, can you please check what's going on? |
Hmm, would the v1.2.x jenkins be hitting an older version of grpc-go that doesn't have the fix from https://github.com/grpc/grpc-go/pull/10309? |
It looks like v1.2.x is still using go1.5: https://github.com/grpc/grpc/blob/v1.2.x/tools/dockerfile/interoptest/grpc_interop_go/Dockerfile#L30. (your link is to grpc-go but I think you meant #10309) |
Just put in the backport PR. |
Not seen recently, closing. |
https://grpc-testing.appspot.com/job/gRPC_interop_pull_requests/9955/console
The text was updated successfully, but these errors were encountered: