-
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
Abuse deployments with extended test #8483
Abuse deployments with extended test #8483
Conversation
c99a5e3
to
83b8a06
Compare
To try locally:
|
[test] |
@ironcladlou can you try this test locally against your instance, trying to verify that it passes for you sometimes (and sometimes it's failing for me because it's unable to access the health check) |
573370a
to
ce43d8b
Compare
[test] |
I was able to trigger a race failure where there are more deployer pods than deployments. Will dig into that more tomorrow. |
b733076
to
83668b8
Compare
|
Looks like we have a race in status latestVersion or something else weird (what code do we have that deletes an RC?) |
Flake #8491 [test] |
83668b8
to
6b5fd75
Compare
6b5fd75
to
7d5dd13
Compare
I saw this one #8483 (comment) and managed to inspect the pod and RC in the cluster- the RC did in fact exist and all the versions aligned correctly for the pod that referenced the RC. I think something must be off in the test causing it to not see the RC. No time to dig further right now, unfortunately. |
@liggitt regarding the fact that we might be querying things that aren't On Wed, Apr 13, 2016 at 2:26 PM, Dan Mace notifications@github.com wrote:
|
[testonlyextended][extended:core(deploymentconfigs)] |
[testonlyextended] On Wed, Apr 13, 2016 at 3:25 PM, OpenShift Bot notifications@github.com
|
7d5dd13
to
6aa26e6
Compare
6aa26e6
to
0b58f70
Compare
[testonlyextended] On Wed, Apr 13, 2016 at 4:21 PM, OpenShift Bot notifications@github.com
|
Evaluated for origin testonlyextended up to 0b58f70 |
continuous-integration/openshift-jenkins/testonlyextended SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin_extended/17/) (Extended Tests: core(deploymentconfigs)) |
Evaluated for origin test up to 0b58f70 |
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/5591/) (Image: devenv-rhel7_3970) |
Evaluated for origin merge up to 0b58f70 |
continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/2982/) (Extended Tests: core(deploymentconfigs)) |
Add a stress test to the cancel / create / deploy logic