-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
wait for kube controllers to be ready before starting informers #17860
wait for kube controllers to be ready before starting informers #17860
Conversation
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, gnufied The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/retest |
3 similar comments
/retest |
/retest |
/retest |
@stevekuznetsov Looks like these router tests are borked. Known problem? |
/retest |
Not known to me @knobunc |
/retest |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
7 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
2 similar comments
/retest |
/retest |
/retest
…On Thu, Jan 4, 2018 at 5:39 AM, OpenShift CI Robot ***@***.*** > wrote:
@deads2k <https://github.com/deads2k>: The following test *failed*, say
/retest to rerun them all:
Test name Commit Details Rerun command
ci/openshift-jenkins/end_to_end 70f5dab
<70f5dab>
link
<https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/17860/test_pull_request_origin_end_to_end/7501/> /test
end_to_end
Full PR test history <https://openshift-gce-devel.appspot.com/pr/17860>. Your
PR dashboard <https://openshift-gce-devel.appspot.com/pr/deads2k>. Please
help us cut down on flakes by linking to
<https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests>
an open issue
<https://github.com/openshift/origin/issues?q=is:issue+is:open> when you
hit one in your PR.
Instructions for interacting with me using PR comments are available here
<https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md>.
If you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
<https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:>
repository. I understand the commands that are listed here
<https://github.com/kubernetes/test-infra/blob/master/commands.md>.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#17860 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAjLVZ02b4519ST05eHJlmx_yKI-ehVHks5tHFYIgaJpZM4RF5gM>
.
|
This needs a green e2e before we merge it. |
/test all |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
1 similar comment
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. |
3.8 pick of #17859