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

Add a bit of extra logging to the dns test when it encounters errors. #9977

Merged
merged 1 commit into from Jun 17, 2015

Conversation

roberthbailey
Copy link
Contributor

No description provided.

@ixdy
Copy link
Member

ixdy commented Jun 17, 2015

LGTM

@ixdy ixdy added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 17, 2015
Do().Raw()
if err != nil {
Do().Raw(); err != nil {
Logf("Unable to read %s from pod %s: %v", fileName, pod.Name, err)
Copy link
Member

Choose a reason for hiding this comment

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

might need a newline at the end?

Copy link
Member

Choose a reason for hiding this comment

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

oh, I guess that's not necessary. OK.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ixdy ixdy added lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed lgtm "Looks good to me", indicates that a PR is ready to be merged. labels Jun 17, 2015
@roberthbailey
Copy link
Contributor Author

Shippable failure seems like a flake, but I don't see a button in their UI to kick the build and make it re-run. Did they remove that recently?

@k8s-bot
Copy link

k8s-bot commented Jun 17, 2015

GCE e2e build/test passed for commit cb7bd71.

saad-ali added a commit that referenced this pull request Jun 17, 2015
Add a bit of extra logging to the dns test when it encounters errors.
@saad-ali saad-ali merged commit dd7a6a0 into kubernetes:master Jun 17, 2015
@roberthbailey roberthbailey deleted the dns-test-error-logging branch August 20, 2015 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants