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

V3-14806 upgrade spanner client #5

Open
wants to merge 1,078 commits into
base: master
Choose a base branch
from

Conversation

petercgrant
Copy link

Upgrades spanner client to upstream 24c5b8f

0xSage and others added 30 commits October 22, 2020 09:35
…leapis#3062)

* docs(logging): update doc link explaining x-cloud-trace-context

* fix(logging): X-Cloud-Trace-Context fields can be optional (traceSample defaults to false)

* style: go fmt logging.go

* style(logging): updated inline comments to be clearer

* refactor(logging): reCloudTraceContext regex is more readable
github.com/googleapis/gapic-generator-go@v0.15.3 from github.com/googleapis/gapic-generator-go/cmd/protoc-gen-go_gapic@v0.15.3 requires github.com/golang/protobuf@v1.4.3
…leapis#3024)

* feature(bigquery): improve error guidance for streaming inserts

This PR attempts to clarify guidance for users of the streaming insert mechanism.

BigQuery enforces a variety of limitations on client requests (size, qps, etc). When
those limits are more severely violated, BigQuery replies with less specific information
about the issue, and instead generates an unstructured HTTP 400 client error response.

This PR adds guidance to doc.go about this behavior, and adds an integration test that
induces these errors to ensure the error surface is behaving as documented.
* feat(spanner): support rw-transaction with options
Implements pubsublite.Client, which wraps the Pub/Sub Lite Admin Service. Includes integration tests for admin operations.
* feat(all): auto-regenerate gapics

This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.

Corresponding genproto PR: googleapis/go-genproto#470
This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.


Corresponding genproto PR: googleapis/go-genproto#471
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/google/pprof](https://togithub.com/google/pprof) | require | digest | `8ef5528` -> `3e6fc7f` |
| [golang.org/x/net](https://togithub.com/golang/net) | require | digest | `f585440` -> `146b70c` |
| [google.golang.org/grpc](https://togithub.com/grpc/grpc-go) | require | minor | `v1.32.0` -> `v1.33.1` |
| [google.golang.org/grpc](https://togithub.com/grpc/grpc-go) | require | patch | `v1.33.0` -> `v1.33.1` |

---

### Release Notes

<details>
<summary>grpc/grpc-go</summary>

### [`v1.33.1`](https://togithub.com/grpc/grpc-go/releases/v1.33.1)

[Compare Source](https://togithub.com/grpc/grpc-go/compare/v1.32.0...v1.33.1)

##### API Changes

-   connectivity: remove unused, experimental Reporter interface ([#&googleapis#8203;3875](https://togithub.com/grpc/grpc-go/issues/3875))

##### New Features

-   xds bootstrap: support insecure and make Creds required ([#&googleapis#8203;3881](https://togithub.com/grpc/grpc-go/issues/3881))
-   xds: add bootstrap support for certificate providers. ([#&googleapis#8203;3901](https://togithub.com/grpc/grpc-go/issues/3901))
-   lrs: add a layer for clusters in load store ([#&googleapis#8203;3880](https://togithub.com/grpc/grpc-go/issues/3880))
-   credentials/xds: implementation of client-side xDS credentials. ([#&googleapis#8203;3888](https://togithub.com/grpc/grpc-go/issues/3888))

##### Bug Fixes

-   http2_client: fix reader segfault on PROTOCOL_ERRORs ([#&googleapis#8203;3926](https://togithub.com/grpc/grpc-go/issues/3926))
    -   Special Thanks: [@&googleapis#8203;sorah](https://togithub.com/sorah)
-   internal/transport: handle h2 errcode on header decoding ([#&googleapis#8203;3872](https://togithub.com/grpc/grpc-go/issues/3872))
    -   Special Thanks: [@&googleapis#8203;tz70s](https://togithub.com/tz70s)
-   xds: exit from run() goroutine when resolver is closed. ([#&googleapis#8203;3882](https://togithub.com/grpc/grpc-go/issues/3882))
-   credentials/alts: ClientAuthorizationCheck to case-fold compare of peer SA ([#&googleapis#8203;3792](https://togithub.com/grpc/grpc-go/issues/3792))
    -   Special Thanks: [@&googleapis#8203;AntonNep](https://togithub.com/AntonNep)
-   service reflection: include transitive closure for a file ([#&googleapis#8203;3851](https://togithub.com/grpc/grpc-go/issues/3851))
-   stats: include message header in stats.InPayload.WireLength ([#&googleapis#8203;3886](https://togithub.com/grpc/grpc-go/issues/3886))
    -   Special Thanks: [@&googleapis#8203;xstephen95x](https://togithub.com/xstephen95x)
-   binarylog: export Sink ([#&googleapis#8203;3879](https://togithub.com/grpc/grpc-go/issues/3879))
-   service reflection: include transitive closure for a file ([#&googleapis#8203;3851](https://togithub.com/grpc/grpc-go/issues/3851))

</details>

---

### Renovate configuration

:date: **Schedule**: "before 3am on Monday" (UTC).

:vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

:recycle: **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

:ghost: **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://togithub.com/renovatebot/config-help/issues) if that's undesired.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/google-cloud-go).
…#3087)

Co-authored-by: Alex Hong <9397363+hongalex@users.noreply.github.com>
…#3086)

Start writting a regen.txt file to go-genproto that records what
hash of googleapis was used to generate the most recent changes.
This is currently not be used, but it will be in the future to
parse commit messages to support context aware commits.
* add readme for pubsublite

* docs: add readme for pubsublite

* update README.md

* update readme
A bucket this test relied upon no longer exists. Refactored test
to not assume objects exist somewhere else. The test now uploads
and cleans-up gcs objects.

Fixes: googleapis#3015
This will be used to manage all of Pub/Sub Lite's bidi streaming RPCs: publish, subscribe, streaming cursor commit, partition assignment, etc.
pubsublite.Message is similar to pubsub.Message, with the following differences:
- Attributes can have multiple values for the same key.
- Pub/Sub Lite uses []byte for data, attribute values and ordering keys.

Message routers select a partition to route a published message to. SHA256 hash is used for routing messages with ordering keys. Round robin is used for routing messages without ordering keys.
This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.


Corresponding genproto PR: googleapis/go-genproto#472
…is#3097)

* docs(logging): add example for logging the HTTPRequest type

* ci(logging): go fmt examples_test.go
Co-authored-by: Cody Oss <6331106+codyoss@users.noreply.github.com>
- Debranded logging and error reporting from "Stackdriver"
- Added api-logging team to codeowners of logging and error reporting
fixes: googleapis#1862 

Changes: 
- No longer panic on User introduced errors. Instead we log the error and let program proceed
- We log.Fatalf("ptypes.TimestampProto(time.Unix(0,0)) failed: %v", err) to exit program, since it's a fatal error/likely not induced by Users
* feat(bigquery): Add support for authorized UDFs

Much like authorized views, BigQuery now supports authorized User
Defined Functions (UDFs). This PR augments the `Access` field of a
dataset entity to support routine references.
This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.


Corresponding genproto PR: googleapis/go-genproto#477
This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.


Corresponding genproto PR: googleapis/go-genproto#478
fixes googleapis#1654 

To de-flake:
- Instead of calling Logs() which is flakey, we write a new log entry then do a single deletion
- Also verified with BE that the quota for all logadmin api calls is 600/min and 1k/hour

It might still flake for a new reason since DeleteLogs is getting deprecated and may be inherently flakey. I will monitor it.
…gleapis#3105)

Added functions to create all the required gapic clients. Updated the AdminClient to use these.
Added utils for attaching headers for routing to the correct backend server handling a particular topic or subscription.
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
tmdiep and others added 26 commits January 14, 2021 20:56
…s#3560)

Return early if there are no messages to wait for. Just check for error, rather than match the error message.

Fixes googleapis#3550.
…apis#3559)

The public API now only accepts resource paths as strings. Structs moved to internal.
Update client to attempt DirectPath by default.

Note that it doesn't mean that after this change client will just use DirectPath, but will call the DirectPath codepath by default.

The actually enablement of DirectPath is controlled by service owner via ACL config. For now, after this change, although all users will attempt DirectPath, but they will all just fallback to the original CFE path.
…leapis#3565)

For consistency with the Pub/Sub Lite Java and Python client libraries.
Co-authored-by: Thiago Nunes <thiagotnunes@google.com>
Sources the default client options from the generated client instead of the hand-written client. This makes sure that new default options that are introduced in the future will automatically be picked up by the Spanner client.

Fixes googleapis#3532
This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.

Corresponding genproto PR: googleapis/go-genproto#516

Changes:

fix!(analytics/admin): remove unused fields from `EnhancedMeasurementSettings` fix!: `update_mask` field is required for all Update operations feat: add pagination support for `ListFirebaseLinks` operation fix!: rename `country_code` field to `region_code` in `Account` fix!: rename `url_query_parameter` field to `uri_query_parameter` in `EnhancedMeasurementSettings` fix!: remove `parent` field from `GoogleAdsLink`
  PiperOrigin-RevId: 351651504
  Source-Link: googleapis/googleapis@23e7356
…pis#3569)

pscompat is more descriptive and indicates that this package is a compatibility API, designed to emulate the Cloud Pub/Sub API.
…rminated (googleapis#3512)

Close gapic clients in SubscriberClient, PublisherClient and tests in order to close client connections. Create new client connections to the mock server in unit tests.
This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.

Corresponding genproto PR: googleapis/go-genproto#517

Changes:

feat(secretmanager): added expire_time and ttl fields to Secret
  PiperOrigin-RevId: 352563582
  Source-Link: googleapis/googleapis@9ecdacc

feat(accessapproval): Migrate the Bazel rules for the libraries published to google-cloud-ruby to use the gapic-generator-ruby instead of the monolith generator.
  PiperOrigin-RevId: 352069361
  Source-Link: googleapis/googleapis@61ab034

chore(kms): un-migrate kms from the Java microgenerator to the monolith
  Committer: @miraleung
  PiperOrigin-RevId: 351867503
  Source-Link: googleapis/googleapis@f1b2cc9
This is an auto-generated regeneration of the gapic clients by
cloud.google.com/go/internal/gapicgen. Once the corresponding genproto PR is
submitted, genbot will update this PR with a newer dependency to the newer
version of genproto and assign reviewers to this PR.

If you have been assigned to review this PR, please:

- Ensure that the version of genproto in go.mod has been updated.
- Ensure that CI is passing. If it's failing, it requires your manual attention.
- Approve and submit this PR if you believe it's ready to ship.


Corresponding genproto PR: googleapis/go-genproto#518

Changes:

feat(secretmanager): added expire_time and ttl fields to Secret
  PiperOrigin-RevId: 352563582
  Source-Link: googleapis/googleapis@9ecdacc
This will help keep branch protection rules up to date on a nightly basis :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.