Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

feat: Add region in Debuggee labels in GCF env #951

Merged
merged 3 commits into from
May 4, 2021

Conversation

Louis-Ye
Copy link
Contributor

  • Read GCF region from either the environment or GCP metadata service
  • Add the region into Debuggee labels
  • Add the corresponding tests

Fixes #950

* Read GCF region from either the environment or GCP metadata service
* Add the region into Debuggee labels
* Add the corresponding tests

Fixes googleapis#950
@Louis-Ye Louis-Ye requested a review from a team as a code owner April 30, 2021 21:14
@product-auto-label product-auto-label bot added the api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. label Apr 30, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Apr 30, 2021
@codecov
Copy link

codecov bot commented Apr 30, 2021

Codecov Report

Merging #951 (80d6287) into master (4992871) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #951      +/-   ##
==========================================
+ Coverage   66.00%   66.02%   +0.02%     
==========================================
  Files          20       20              
  Lines        1606     1619      +13     
  Branches      330      334       +4     
==========================================
+ Hits         1060     1069       +9     
- Misses        470      472       +2     
- Partials       76       78       +2     
Impacted Files Coverage Δ
src/agent/debuglet.ts 80.64% <100.00%> (-0.39%) ⬇️

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 4992871...80d6287. Read the comment docs.

gugupopes123 referenced this pull request Apr 30, 2021
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
src/agent/debuglet.ts Outdated Show resolved Hide resolved
src/agent/debuglet.ts Outdated Show resolved Hide resolved
src/agent/debuglet.ts Outdated Show resolved Hide resolved
src/agent/debuglet.ts Outdated Show resolved Hide resolved
src/agent/debuglet.ts Outdated Show resolved Hide resolved
@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 4, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 4, 2021
Copy link
Contributor

@bcoe bcoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small structural nit, which I think might simplify logic a bit.

Otherwise looks good to me once @mctavish approves.

src/agent/debuglet.ts Show resolved Hide resolved
@Louis-Ye Louis-Ye requested a review from mctavish May 4, 2021 14:01
@mctavish mctavish merged commit a88e904 into googleapis:master May 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add region in the Debuggee labels for GCF envionrment
4 participants