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.

Sign up for GitHub

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

Jump to bottom

add a test for fetchruntimeartifacts failure #12467

Merged
merged 1 commit into from
Jan 16, 2017
Merged

add a test for fetchruntimeartifacts failure #12467

merged 1 commit into from
Jan 16, 2017

Conversation

PI-Victor
Copy link
Contributor

  • rename one variable to upercase URL

@PI-Victor
Copy link
Contributor Author

PI-Victor commented Jan 12, 2017 •
edited by bparees
Loading

[test]
[testextended][extended:core(update failure status)]
cc: @bparees

@PI-Victor PI-Victor force-pushed the add_runtimeartifactsfailure_test branch from 494a3d9 to 5bc05bd Compare January 13, 2017 07:41
@PI-Victor PI-Victor mentioned this pull request Jan 13, 2017
Closed
@PI-Victor
Copy link
Contributor Author

*sigh
flake : #12479

https://ci.openshift.redhat.com/jenkins/job/test_pr_origin_extended/985/consoleFull#123465095256cbb9a5e4b02b88ae8c2f77

the test. however. passes:

• [SLOW TEST:27.325 seconds]
[builds][Slow] update failure status
/data/src/github.com/openshift/origin/test/extended/builds/failure_status.go:159
  Build status failed assemble runtime artifacts 
  /data/src/github.com/openshift/origin/test/extended/builds/failure_status.go:158
    should contain the failure reason related to failing to fetch runtime image artifacts
    /data/src/github.com/openshift/origin/test/extended/builds/failure_status.go:157
------------------------------
SSSSSSSSS

@bparees
Copy link
Contributor

bparees commented Jan 16, 2017

squash please

@bparees
Copy link
Contributor

bparees commented Jan 16, 2017

i've updated your comment to just run your new test.

@bparees bparees self-assigned this Jan 16, 2017
@PI-Victor PI-Victor force-pushed the add_runtimeartifactsfailure_test branch from 4e91c00 to a6c73fc Compare January 16, 2017 19:04
@PI-Victor
Copy link
Contributor Author

@bparees squashed. forgot you could filter specific tests as well, thanks!

@openshift-bot
Copy link
Contributor

Evaluated for origin testextended up to a6c73fc

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to a6c73fc

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/testextended SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin_extended/987/) (Base Commit: 9478e0b) (Extended Tests: core(update failure status))

@bparees
Copy link
Contributor

bparees commented Jan 16, 2017

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Jan 16, 2017 •
edited
Loading

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/12912/) (Image: devenv-rhel7_5690)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to a6c73fc

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/12912/) (Base Commit: 9478e0b)

@openshift-bot openshift-bot merged commit 9b3acf5 into openshift:master Jan 16, 2017
@PI-Victor PI-Victor deleted the add_runtimeartifactsfailure_test branch January 16, 2017 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bparees bparees

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PI-Victor @bparees @openshift-bot