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.

Sign up for GitHub

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

Jump to bottom

GCE provider: Log full contents of long operations #26962

Merged
merged 1 commit into from
Jun 12, 2016
Merged

GCE provider: Log full contents of long operations #26962

merged 1 commit into from
Jun 12, 2016
+16 −2

Conversation

zmerlynn
Copy link
Member

@zmerlynn zmerlynn commented Jun 7, 2016

AnalyticsDump JSON of long running (>1m) GCE operations.

@zmerlynn
GCE provider: Log full contents of long operations
ce6537a 
Dump JSON of long running (>1m) GCE operations.
@zmerlynn zmerlynn added area/platform/gce release-note Denotes a PR that will be considered when it comes time to generate release notes. labels Jun 7, 2016
@k8s-github-robot k8s-github-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 7, 2016
@zmerlynn
Copy link
Member Author

zmerlynn commented Jun 7, 2016

@k8s-bot e2e test this, issue #IGNORE (PR builder flake)

@wojtek-t wojtek-t added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 7, 2016
@wojtek-t
Copy link
Member

wojtek-t commented Jun 7, 2016

LGTM - thanks!

@zmerlynn
Copy link
Member Author

zmerlynn commented Jun 7, 2016

Looks like we're blocked on #23545 :(

@zmerlynn zmerlynn added this to the v1.3 milestone Jun 7, 2016
@fgrzadkowski
Copy link
Contributor

@k8s-bot test this issue: #23545

@zmerlynn zmerlynn added the priority/backlog Higher priority than priority/awaiting-more-evidence. label Jun 7, 2016
@zmerlynn
Copy link
Member Author

zmerlynn commented Jun 7, 2016

@k8s-bot test this, issue #IGNORE (early PR builder flake, fixed now)

@zmerlynn
Copy link
Member Author

zmerlynn commented Jun 7, 2016

@k8s-bot node e2e test this issue: #26431

@zmerlynn zmerlynn removed the priority/backlog Higher priority than priority/awaiting-more-evidence. label Jun 7, 2016
@k8s-bot
Copy link

k8s-bot commented Jun 12, 2016

GCE e2e build/test passed for commit ce6537a.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 5d87101 into kubernetes:master Jun 12, 2016
@zmerlynn
Copy link
Member Author

This was in the queue for almost a week, and I'd really like it for diagnostics on long-running GCE operations. Requesting cherry-pick.

@zmerlynn zmerlynn deleted the log-gce-better branch June 13, 2016 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@wojtek-t wojtek-t

Labels
area/platform/gce lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
v1.3
Development

Successfully merging this pull request may close these issues.
7 participants
@zmerlynn @wojtek-t @fgrzadkowski @k8s-bot @k8s-github-robot @googlebot @eparis