Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

Update kube-dns and switch sidecar to probe SRV records #830

Merged
merged 2 commits into from
Jan 24, 2018
Merged

Update kube-dns and switch sidecar to probe SRV records #830

merged 2 commits into from
Jan 24, 2018

Conversation

dghubble
Copy link
Contributor

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 12, 2018
@dghubble
Copy link
Contributor Author

Jenkins was down, re-running tests.

@dghubble
Copy link
Contributor Author

coreosbot run e2e

@dghubble
Copy link
Contributor Author

coreosbot run e2e calico

@dghubble
Copy link
Contributor Author

coreosbot run e2e

@rphillips
Copy link
Contributor

This is plagued by the same TestSmoke flake.

@dghubble
Copy link
Contributor Author

I've seen both e2e and e2e-calico succeed on independent runs, but yeah, its the TestSmoke flake.

@xiang90
Copy link
Contributor

xiang90 commented Jan 12, 2018

@rphillips @dghubble

I suggest us to get PR merged if they are blocked by any known flake, and we believe this PR wont make the flake more serious.

@ericchiang
Copy link
Contributor

Same failure we saw in #828 (comment), that's not good.

@dghubble
Copy link
Contributor Author

Gah, forgot to merge this :(

@dghubble
Copy link
Contributor Author

coreosbot run e2e

@dghubble
Copy link
Contributor Author

dghubble commented Jan 22, 2018

Passed tests, but rebasing to the history is cleaner.

@dghubble
Copy link
Contributor Author

coreosbot run e2e

@dghubble
Copy link
Contributor Author

coreosbot run e2e

1 similar comment
@ericchiang
Copy link
Contributor

coreosbot run e2e

@ericchiang
Copy link
Contributor

@dghubble tests are green. feel free to merge

@dghubble
Copy link
Contributor Author

Finally!

@dghubble dghubble merged commit 6b9ca20 into kubernetes-retired:master Jan 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants