-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat(transport): add google-c2p dependence to DirectPath (#1260)
DirectPath is migrating to Traffic Director, and we want to setup E2E tests based on TD for Bigtable and Spanner. This PR: 1. Update grpc-go version to 1.41.0, which has RING_HASH LB policy support; 2. Add the google-c2p resolver dependence to google-api-go-client; 3. Add an environment variable GOOGLE_CLOUD_ENABLE_DIRECT_PATH_XDS to enable using DirectPath over TD; Corresponding PR in gax-java: googleapis/gax-java#1521. Also note that currently there is a bug in grpc-go, and fix is grpc/grpc-go#4889. I have verified that with this fix Bigtable E2E test over DirectPath Traffic Director can pass. This PR has been merged to grpc-go and will release in grpc-go 1.42.0, which will be the minimal version for supporting `google-c2p` URI scheme.
- Loading branch information
1 parent
0ac6baa
commit aa0f0be
Showing
3 changed files
with
62 additions
and
23 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| // Copyright 2021 Google LLC. | ||
| // Use of this source code is governed by a BSD-style | ||
| // license that can be found in the LICENSE file. | ||
|
|
||
| // Package grpc supports network connections to GRPC servers. | ||
| // This package is not intended for use by end developers. Use the | ||
| // google.golang.org/api/option package to configure API clients. | ||
|
|
||
| //go:build enablexds | ||
| // +build enablexds | ||
|
|
||
| package grpc | ||
|
|
||
| import ( | ||
| // Install google-c2p resolver, which is required for direct path. | ||
| _ "google.golang.org/grpc/xds/googledirectpath" | ||
| ) |