Skip to content

Conversation

@cdrage
Copy link
Member

@cdrage cdrage commented Jun 8, 2016

We create the new project a bit too fast before OpenShift can register
it in the background. This should fix the failed test encountered in:
https://ci.centos.org/job/atomicapp-test-openshift-pr/95/consoleFull

@cdrage
Copy link
Member Author

cdrage commented Jun 9, 2016

ping @dustymabe

@dustymabe
Copy link
Contributor

@cdrage - can we add a comment to the code about why we think this is needed?

We create the new project a bit too fast before OpenShift can register
it in the background. This should fix the failed test encountered in:
https://ci.centos.org/job/atomicapp-test-openshift-pr/95/consoleFull
@cdrage cdrage force-pushed the fix-timing-issue-openshift branch from 6f8717c to 640fa36 Compare June 9, 2016 13:02
@cdrage
Copy link
Member Author

cdrage commented Jun 9, 2016

@dustymabe done :)

@dustymabe
Copy link
Contributor

preferably we wouldn't have to do this, but YOLO
LGTM

@cdrage
Copy link
Member Author

cdrage commented Jun 9, 2016

YOLO MERGE 🚗

@cdrage cdrage merged commit 4b2feb3 into projectatomic:master Jun 9, 2016
@cdrage
Copy link
Member Author

cdrage commented Jun 9, 2016

See if this fixes everythin' :) Running a test now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants