-
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
SDN test should wait longer for namespaces #15700
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: smarterclayton 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 |
@deads2k integration is continually flaking, this is extracted from my integration refactor where it stomped that |
@smarterclayton: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
e2e failed building images but didn't exit: #15702 |
Passed integration. Force merging. |
You think that non-parallel it takes more than two seconds to observe a create? |
Fairly sure - but the retry logic is not properly printing out the second
run logs correctly.
…On Wed, Aug 9, 2017 at 12:32 PM, David Eads ***@***.***> wrote:
@deads2k <https://github.com/deads2k> integration is continually flaking,
this is extracted from my integration refactor where it stomped that
You think that non-parallel it takes more than two seconds to observe a
create?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#15700 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABG_pzzUIE3n5EfVCtXzuM93mqHqGqM0ks5sWd8dgaJpZM4OyLPM>
.
|
It's not just failing to observe the creation of the newly-created namespace. It doesn't even observe the existence of "default". Maybe something is causing startup of the shared informer to be unexpectedly slow? |
Hard to believe that would happen just this informer/handler pair and never another or at least not to anything approaching the same degree |
Note that some of the runs were from before the fix to the start hook
landed. I'm still watching for the second set of hooks.
…On Wed, Aug 9, 2017 at 1:39 PM, David Eads ***@***.***> wrote:
It's not just failing to observe the creation of the newly-created
namespace. It doesn't even observe the existence of "default". Maybe
something is causing startup of the shared informer to be unexpectedly slow?
Hard to believe that would happen just *this* informer/handler pair and
never another or at least not to anything approaching the same degree
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#15700 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABG_p0U31T_Fsii5q-R6Hz4MPMbF0DArks5sWe7bgaJpZM4OyLPM>
.
|
Might be a hook init failure:
https://storage.googleapis.com/origin-ci-test/pr-logs/pull/15248/test_pull_request_origin_integration/5706/build-log.txt
or
could be the latent flake we have in project watch test.
After this merged I saw no more oadmpodnetwork flakes.
|
Flaking heavily in master