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

fix: change clusterId in Cmek test #716

merged 2 commits into from Apr 9, 2021


Copy link

@ad548 ad548 commented Apr 9, 2021

Change-Id: Idc8ad885205190374b089349ee94cc627bafd782

Getting an error internally about cluster with clusterId2 already existing. So changing to use new clusterId for the other zone scenario.

@ad548 ad548 requested a review from as a code owner Apr 9, 2021
@google-cla google-cla bot added the cla: yes label Apr 9, 2021
@product-auto-label product-auto-label bot added the api: bigtable label Apr 9, 2021
Change-Id: Idc8ad885205190374b089349ee94cc627bafd782
Change-Id: I8d0ce18de5fa8e914344340850786930fbd670f1
@ad548 ad548 changed the title change clusterId in Cmek test fix: change clusterId in Cmek test Apr 9, 2021
Copy link

@codecov codecov bot commented Apr 9, 2021

Codecov Report

Merging #716 (21e3d1b) into master (da919e1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #716   +/-   ##
  Coverage     83.64%   83.64%           
  Complexity     1220     1220           
  Files           112      112           
  Lines          7417     7417           
  Branches        394      394           
  Hits           6204     6204           
  Misses         1010     1010           
  Partials        203      203           

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 da919e1...21e3d1b. Read the comment docs.

kolea2 approved these changes Apr 9, 2021
@kolea2 kolea2 merged commit 67ef155 into googleapis:master Apr 9, 2021
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
api: bigtable cla: yes
None yet

Successfully merging this pull request may close these issues.

None yet

2 participants