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

github.com/openshift/origin/test/integration .TestOadmPodNetwork fails #15658

Closed
deads2k opened this issue Aug 7, 2017 · 4 comments
Closed
Assignees
Labels
component/networking kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1

Comments

@deads2k
Copy link
Contributor

deads2k commented Aug 7, 2017

I've seen this in several jobs:

https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_integration/5502/
https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_integration/5538

@deads2k deads2k added the kind/test-flake Categorizes issue or PR as related to test flakes. label Aug 7, 2017
@deads2k
Copy link
Contributor Author

deads2k commented Aug 9, 2017

seven out of the top 10 failed integration tests are failing on this: https://deck-ci.svc.ci.openshift.org/?job=test_pull_request_origin_integration&state=failure. Making it a p1

@danwinship @dcbw @knobunc I'm not sure who owns it, but it appear network-y.

@danwinship
Copy link
Contributor

In #15336 (at the bottom), @liggitt said that this was actually caused by #15648, which was fixed by #15654 yesterday morning. It looks like this failure has still occurred since that merged, although I can't figure out how to tell the exact master commit a given test run merged against.


Assuming the problem isn't fixed, it's because the Namespace SharedInformer doesn't seem to be informing about any Namespaces. In a successful run there are logs like:

I0809 10:02:17.079663   31523 vnids_master.go:290] Watch ADDED event for Namespace "one"
I0809 10:02:17.080492   31523 vnids_master.go:122] Allocated netid 13639912 for namespace "one"

but those are missing in the logs of the failed test runs

@liggitt
Copy link
Contributor

liggitt commented Aug 9, 2017

fixed by #15700?

@danwinship
Copy link
Contributor

fixed by #15700?

yup, apparently

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/networking kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1
Projects
None yet
Development

No branches or pull requests

6 participants