Skip to content
This repository has been archived by the owner on Dec 8, 2021. It is now read-only.

feat: environment-variable override of the spanner endpoint #1269

Merged
merged 1 commit into from
Feb 18, 2020

Conversation

devbww
Copy link
Contributor

@devbww devbww commented Feb 18, 2020

When the SPANNER_EMULATOR_HOST environment variable is set, the
endpoint and credentials for data and admin connections are overridden
by ${SPANNER_EMULATOR_HOST} and grpc::InsecureChannelCredentials()
respectively.


This change is Reviewable

When the `SPANNER_EMULATOR_HOST` environment variable is set, the
endpoint/credentials for data and admin connections are overridden
by `${SPANNER_EMULATOR_HOST}` and `grpc::InsecureChannelCredentials()`
respectively.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 18, 2020
@codecov
Copy link

codecov bot commented Feb 18, 2020

Codecov Report

Merging #1269 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1269      +/-   ##
==========================================
+ Coverage    94.8%   94.85%   +0.05%     
==========================================
  Files         181      181              
  Lines       13789    13827      +38     
==========================================
+ Hits        13073    13116      +43     
+ Misses        716      711       -5
Impacted Files Coverage Δ
google/cloud/spanner/internal/spanner_stub_test.cc 100% <ø> (ø) ⬆️
google/cloud/spanner/connection_options_test.cc 100% <100%> (ø) ⬆️
google/cloud/spanner/connection_options.cc 97.61% <100%> (+0.32%) ⬆️
google/cloud/spanner/samples/samples.cc 87.94% <100%> (+0.37%) ⬆️
...tegration_tests/database_admin_integration_test.cc 94.11% <100%> (+0.24%) ⬆️
google/cloud/spanner/connection_options.h 100% <100%> (ø) ⬆️
...anner/integration_tests/client_integration_test.cc 98.48% <100%> (+0.01%) ⬆️
...tegration_tests/instance_admin_integration_test.cc 86.81% <100%> (+0.44%) ⬆️
google/cloud/spanner/keys.h 100% <0%> (ø) ⬆️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45961b8...d23b2b0. Read the comment docs.

@devbww devbww requested a review from devjgm February 18, 2020 20:58
@devbww devbww marked this pull request as ready for review February 18, 2020 20:58
@devbww devbww merged commit 5270018 into googleapis:master Feb 18, 2020
@devbww devbww deleted the emulator branch February 18, 2020 23:00
devjgm pushed a commit to devjgm/google-cloud-cpp that referenced this pull request May 7, 2020
…is/google-cloud-cpp-spanner#1269)

When the `SPANNER_EMULATOR_HOST` environment variable is set, the
endpoint/credentials for data and admin connections are overridden
by `${SPANNER_EMULATOR_HOST}` and `grpc::InsecureChannelCredentials()`
respectively.

Fixes googleapis/google-cloud-cpp-spanner#1200.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants