gapic-generator-ruby

gapic-generator-ruby copied to clipboard

This appears in `google-cloud-firestore-admin-v1/proto_docs/google/firestore/admin/v1/location.rb` It is currently converting `[google.cloud.location.Location.metadata][google.cloud.location.Location.metadata]` to `{::Google::Cloud::Location::Location#metadata google.cloud.location.Location.metadata}`, but the class is defined in a dependency (google-cloud-location); hence YARD tests fail. (We are currently working around...

dazuma

This is brought on by https://github.com/googleapis/google-cloud-ruby/issues/14050 The end-user does not know about the default per-rpc timeout being generated, so they just percieve the client-wide timeout to 'not have any effect'....

viacheslav-rostovtsev

Document how to access trailing_metadata from ActiveCall::Operation on streaming responses

5

Currently the underlying `GRPC::ActiveCall::Operation` is exposed to users directly for every method call and it is the only way to access some gRPC properties, like the `trailing_metadata`. This could be...

jbolinger

feat(gapic-common): Add retry options and expanded retry logic

1

The [google-cloud-pubsub](https://github.com/googleapis/google-cloud-ruby/blob/master/google-cloud-pubsub) library is required to support the following RPC retry policy, provided by the Pub/Sub team: | property | default value | | ---------------------- | ------ | | init...

quartzmo

Include `ruby-cloud-generate-standalone-snippets` and `ruby-cloud-generate-yardoc-snippets` in `shared/test_resources/api_test_resources.rb`, and in fact we should probably turn them on for all tests and embedded outputs at this point. Note this may depend on #629.

dazuma

Rework the unit tests for diregapic so that the test harness does not duplicate helper methods from gapic-common/rest

viacheslav-rostovtsev

OAuth errors (e.g. from invalid token) surface as GRPC::Unavailable, but some of them (e.g. Signet:AuthorizationError on presenting an expired refresh token) are clearly unretriable. Should something be done about this?

4

https://github.com/googleads/google-ads-ruby/issues/246 for details. The question is whether error wrapping is working as intended.

viacheslav-rostovtsev

fix(spanner): convert enumerable GRPC::BadStatus errors to Google::Cloud::Errors

4

TLDR: we would like to have the conversion of `GRPC::BadStatus` errors to `Google::Cloud::Error`s in the `Enumerable` result of streaming calls of the spanner client, so that the consumer only has...

thiagotnunes

Improve the proto docs to reflect the connection between the oneof fields

1

Protos allow some fields to be connected in what essentially are the oneof groups. Assigning one field in the group will essentially prevent others from being assigned and there might...

viacheslav-rostovtsev

Currently IIRC the logic is 'every argument except the last' or something close but should take segment ids with/without patterns into an account

viacheslav-rostovtsev