Darren Bolduc
Darren Bolduc
Checkpointing progress... I think I am finally over the hump with this feature. https://github.com/dbolduc/google-cloud-rust/commits/additional-bindings-2-dev This branch implements additional bindings / path validation for some of the services in our integration...
Another day of progress. I factored out the errors and supported non-string fields. The generated code looks like: https://github.com/dbolduc/google-cloud-rust/blob/a4c36bc7358a3cbe4aad472f2b9cfd7a3ff31973/src/generated/cloud/secretmanager/v1/src/transport.rs#L456-L544 I can run the integration tests (including gapic-showcase) for real this...
> additional bindings for mixins will be non-trivial. The LRO methods aggregate all of the additional bindings across all of the services. I was wrong. > There are ~90 of...
Pushed another commit that simplified the path construction. No more local variables. They do not save us anything. And another commit to support query parameters *per binding*. Note that "supporting...
Here is the execution plan (each task is roughly a PR): ## Gax changes - [x] Introduce `BindingError` et al. in `gax` - with fmt functions - [x] Introduce `PathMismatchBuilder`...
Should we add user project to `RequestOptions`, `RequestOptionsBuilder` ? https://github.com/googleapis/google-cloud-rust/blob/e40933ae224e4f7db2c0aadd6bb1f70216958e18/src/gax/src/options.rs#L43 and handle it in the `gaxi` clients https://github.com/googleapis/google-cloud-rust/blob/e40933ae224e4f7db2c0aadd6bb1f70216958e18/src/gax-internal/src/grpc.rs#L215 https://github.com/googleapis/google-cloud-rust/blob/e40933ae224e4f7db2c0aadd6bb1f70216958e18/src/gax-internal/src/http.rs#L96
FYI: @nvnmandadhi This will involve changes to the generator, which is in the `librarian` repo: https://github.com/googleapis/librarian/ You will find this guide useful: https://github.com/googleapis/google-cloud-rust/blob/main/doc/contributor/howto-guide-generated-code-maintenance.md#making-changes-to-sidekick --- This is what the OCC loop...
Unassigning because I am not actively working on this. I may get to it the next time I am on-duty.
The code in this section should look similar to these unit tests: https://github.com/googleapis/google-cloud-rust/blob/29eec8c0027fd4449bec3e4ea5dff67e5968fd0b/src/integration-tests/tests/pagination.rs
It took me several iterations to get a `create_bucket` request to work. https://github.com/googleapis/google-cloud-rust/blob/57a698a58b133d43c2746c25d4c343c5d95909f2/src/integration-tests/src/storage.rs#L47-L55 The result does not look much like our sample.