-
Notifications
You must be signed in to change notification settings - Fork 4
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
Don't wait for Cromwell Label endpoint to return #953
Conversation
@@ -208,5 +209,76 @@ class RawlsApiSpec extends FreeSpec with Matchers with Eventually | |||
} | |||
|
|||
} | |||
|
|||
"should retrieve metadata with widely scattered sub-workflows in a short time" in { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adapted from the previous test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 code looks good; assuming the test works as planned.
on hold pending decisions |
- partial solution for DataBiosphere/firecloud-app#97
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
jenkins restart |
jenkins retest |
Jenkins failures are all spurious "GPAlloc project already exists" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on our conversation, please improve the text of an error message displayed to users
This reverts commit 9c5ff47.
Stopgap solution for DataBiosphere/firecloud-app#97 with longer-term fix to follow
dev
and
alpha
DBs in Google Cloud Console