Skip to content
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(deps): bump google.golang.org/grpc from 1.33.2 to 1.34.0 #8162

Closed

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Dec 14, 2020

Bumps google.golang.org/grpc from 1.33.2 to 1.34.0.

Release notes

Sourced from google.golang.org/grpc's releases.

Release 1.34.0

New Features

  • client: implement support for "unix" resolver scheme (#3890)
  • rds: allow case_insensitive path matching (#3997)
  • credentials/insecure: implement insecure credentials. (#3964)
  • lrs: handle multiple clusters in LRS stream (#3935)

Performance Improvements

  • encoding/proto: simplify & optimize proto codec (#3958)

Bug Fixes

  • internal/transport: fix a bug causing -bin metadata to be incorrectly encoded (#3985)
  • grpclb: consider IDLE SubConns as connecting (#4031)
  • grpclb: send custom user-agent (#4011)
  • client: use "localhost:port" as authority if target is ":port" (#4017)
  • credentials: fix PerRPCCredentials w/RequireTransportSecurity and security levels (#3995)

Documentation

  • Documentation: fix outgoing metadata example code (#3979)
  • Remove experimental comment from client interceptors (#3948)
  • Documentation: update keepalive.md to add why (#3993)
Commits
  • 2cf12a8 Change version to 1.34.0 (#4042)
  • f5c42ca xds/client: Export a method to return bootstrap config. (#4033)
  • b88744b xds: add ConfigSelector to support RouteAction timeouts (#3991)
  • 20636e7 protobuf: update protoc to latest version (#4038)
  • 707e298 internal: fix net.Dial fail without IP (#4037)
  • 1d01bf9 grpclb: consider IDLE SubConns as connecting (#4031)
  • 90f1b3e client: use "localhost:port" as authority if target is ":port" (#4017)
  • 22dba5e xds: Remove usages of grpc.WithInsecure(). (#4028)
  • d2629bd xds/bootstrap: Add support for grpc_server_resource_name_id. (#4030)
  • 6caf9d8 xds: make xds client a singleton (#4015)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added A:automerge Automatically merge PR once all prerequisites pass. dependencies Pull requests that update a dependency file labels Dec 14, 2020
@dependabot dependabot bot requested a review from fedekunze December 14, 2020 07:00
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.33.2 to 1.34.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.33.2...v1.34.0)

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot force-pushed the dependabot/go_modules/google.golang.org/grpc-1.34.0 branch from 5f0d116 to ef206d5 Compare December 15, 2020 12:39
@amaury1093
Copy link
Contributor

Some tests panic here:

// proto marshal the result bytes
resBytes, err := protoCodec.Marshal(res)

and that's because res (e.g. QueryAnnualProvisionsResponse) contains a subfield that's a customtype (e.g. sdk.Dec).

I wonder if it's just a go-grpc<->gogoproto compatibility issue.

Copy link
Contributor

@alessio alessio left a comment

Choose a reason for hiding this comment

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

Unfortunately, tests still fail ☹️

@alessio alessio removed the A:automerge Automatically merge PR once all prerequisites pass. label Dec 21, 2020
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Jan 12, 2021

A newer version of google.golang.org/grpc exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

@amaury1093 amaury1093 mentioned this pull request Jan 19, 2021
9 tasks
@fedekunze
Copy link
Collaborator

can someone update this?

@fdymylja
Copy link
Contributor

fdymylja commented Jan 19, 2021

After some tests, the panic comes from grpc calling google.golang.org/protobuf@1.25.0

perhaps a module replace might fix this, but I'd have to test (NVM actually due to module path naming differences)

Stack trace:

google.golang.org/protobuf/internal/impl.newSingularConverter at convert.go:143
google.golang.org/protobuf/internal/impl.NewConverter at convert.go:60
google.golang.org/protobuf/internal/impl.fieldInfoForScalar at message_reflect_field.go:234
google.golang.org/protobuf/internal/impl.(*MessageInfo).makeKnownFieldsFunc at message_reflect.go:67
google.golang.org/protobuf/internal/impl.(*MessageInfo).makeReflectFuncs at message_reflect.go:36
google.golang.org/protobuf/internal/impl.(*MessageInfo).initOnce at message.go:90
google.golang.org/protobuf/internal/impl.(*MessageInfo).init at message.go:72
google.golang.org/protobuf/internal/impl.(*messageReflectWrapper).ProtoMethods at message_reflect_gen.go:150
google.golang.org/protobuf/proto.protoMethods at proto_methods.go:18
google.golang.org/protobuf/proto.MarshalOptions.marshal at encode.go:143
google.golang.org/protobuf/proto.MarshalOptions.MarshalAppend at encode.go:125
github.com/golang/protobuf/proto.marshalAppend at wire.go:40
github.com/golang/protobuf/proto.Marshal at wire.go:23
google.golang.org/grpc/encoding/proto.codec.Marshal at proto.go:39
<autogenerated>:2
github.com/cosmos/cosmos-sdk/baseapp.(*GRPCQueryRouter).RegisterService.func1 at grpcrouter.go:99
github.com/cosmos/cosmos-sdk/baseapp.(*BaseApp).handleQueryGRPC at abci.go:562
github.com/cosmos/cosmos-sdk/baseapp.(*BaseApp).Query at abci.go:406
github.com/tendermint/tendermint/abci/client.(*localClient).QuerySync at local_client.go:250
github.com/tendermint/tendermint/proxy.(*appConnQuery).QuerySync at app_conn.go:159
github.com/tendermint/tendermint/rpc/core.ABCIQuery at abci.go:20
github.com/tendermint/tendermint/rpc/client/local.(*Local).ABCIQueryWithOptions at local.go:87
github.com/cosmos/cosmos-sdk/client.Context.queryABCI at query.go:76
github.com/cosmos/cosmos-sdk/client.Context.QueryABCI at query.go:52
github.com/cosmos/cosmos-sdk/client.Context.Invoke at grpc_query.go:84
<autogenerated>:2
github.com/cosmos/cosmos-sdk/x/mint/types.(*queryClient).AnnualProvisions at query.pb.go:362
github.com/cosmos/cosmos-sdk/x/mint/client/cli.GetCmdQueryAnnualProvisions.func1 at query.go:108
github.com/spf13/cobra.(*Command).execute at command.go:850
github.com/spf13/cobra.(*Command).ExecuteC at command.go:958
github.com/spf13/cobra.(*Command).Execute at command.go:895
github.com/spf13/cobra.(*Command).ExecuteContext at command.go:888
github.com/cosmos/cosmos-sdk/testutil/cli.ExecTestCLICmd at cmd.go:22
github.com/cosmos/cosmos-sdk/x/mint/client/cli_test.(*IntegrationTestSuite).TestGetCmdQueryAnnualProvisions.func1 at cli_test.go:159
github.com/stretchr/testify/suite.(*Suite).Run.func1 at suite.go:77
testing.tRunner at testing.go:1123
runtime.goexit at asm_amd64.s:1374
 - Async stack trace
testing.(*T).Run at testing.go:1168

@amaury1093
Copy link
Contributor

Can I close this? I believe the correct fix is to update to go proto v2. There's an issue tracking this particular dep bump: #8392

@amaury1093 amaury1093 closed this Feb 22, 2021
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 22, 2021

OK, I won't notify you again about this release, but will get in touch when a new version is available.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@amaury1093 amaury1093 deleted the dependabot/go_modules/google.golang.org/grpc-1.34.0 branch February 22, 2021 12:14
@amaury1093 amaury1093 mentioned this pull request Mar 30, 2021
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants