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

Add debugger support for Cloud Run. See b/112087600 for context#671

Merged
DominicKramer merged 4 commits intogoogleapis:masterfrom
hrasadi:master
Apr 30, 2019
Merged

Add debugger support for Cloud Run. See b/112087600 for context#671
DominicKramer merged 4 commits intogoogleapis:masterfrom
hrasadi:master

Conversation

@hrasadi
Copy link
Copy Markdown
Contributor

@hrasadi hrasadi commented Apr 16, 2019

(Partially) Fixes b/112087600.

  • [✓] Tests and linter pass
  • [✓] Code coverage does not decrease (if any source code was changed)
  • [✓] Appropriate docs were updated (if necessary)

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 16, 2019
Copy link
Copy Markdown
Contributor

@DominicKramer DominicKramer left a comment

Choose a reason for hiding this comment

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

Thanks for creating this PR. It is much appreciated. I think it looks good to me with one comment. I think the approach you took is correct, but I just want to make sure.

@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 18, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 18, 2019
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 18, 2019

Codecov Report

Merging #671 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #671   +/-   ##
=======================================
  Coverage   89.17%   89.17%           
=======================================
  Files          13       13           
  Lines         776      776           
  Branches       66       66           
=======================================
  Hits          692      692           
  Misses         67       67           
  Partials       17       17
Impacted Files Coverage Δ
src/agent/debuglet.ts 83.6% <100%> (ø) ⬆️

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 c506279...ac546fb. Read the comment docs.

@ofrobots
Copy link
Copy Markdown
Contributor

/cc @kjin @nolanmar511: the same may be needed on Trace and Profiler.

@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Apr 23, 2019
@ofrobots ofrobots added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 23, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 23, 2019
@DominicKramer DominicKramer added status: blocked Resolving the issue is dependent on other work. and removed 🚨 This issue needs some love. labels Apr 26, 2019
@DominicKramer DominicKramer added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2019
@DominicKramer DominicKramer added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 30, 2019
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 30, 2019
@DominicKramer DominicKramer merged commit 505cbc9 into googleapis:master Apr 30, 2019
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. status: blocked Resolving the issue is dependent on other work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants