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

Revert "Use service name as debuggee id on gke (#275)"#278

Merged
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:revert-275
Jun 19, 2017
Merged

Revert "Use service name as debuggee id on gke (#275)"#278
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:revert-275

Conversation

@ofrobots
Copy link
Copy Markdown
Contributor

This reverts commit d3994f8 as it breaks on non-GKE GCP enviornments.

See: #275 (comment)

This reverts commit d3994f8 as it
breaks on non-GKE GCP enviornments.

See: googleapis#275 (comment)
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 19, 2017
@ofrobots
Copy link
Copy Markdown
Contributor Author

TBR= @matthewloring

@ofrobots ofrobots merged commit 66d67f8 into googleapis:master Jun 19, 2017
Copy link
Copy Markdown
Contributor

@matthewloring matthewloring left a comment

Choose a reason for hiding this comment

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

LGTM

DominicKramer added a commit to DominicKramer/cloud-debug-nodejs that referenced this pull request Jun 23, 2017
commit 69106322b587bc6152199cdc0ec9b70f47afba03
Author: Dominic Kramer <[email protected]>
Date:   Mon Jun 19 13:51:32 2017 -0700

    Convert debuglet.ts to use Typescript classes

    The conversion now includes the changes in PR googleapis#278 that reverts
    PR googleapis#275 (Use service name as debuggee id on gke).

commit afa5e7e5a5df2534e9265f01df61aed6d8b11327
Author: Dominic Kramer <[email protected]>
Date:   Mon Jun 19 13:48:46 2017 -0700

    Rename debuglet.js to debuglet.ts
@ofrobots ofrobots deleted the revert-275 branch July 6, 2017 18:09
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