-
Notifications
You must be signed in to change notification settings - Fork 1
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
Pivotal ID # 187734050: CLI Submit Timeout #847
Pivotal ID # 187734050: CLI Submit Timeout #847
Conversation
- Resource to get a request status - Move the RequestStatus to the core model - Add a new parameter to specify the timeout on submit - Add missing deployment stage - Fix CLI docs - Remove unnecessary properties from the config files
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.
Lookin good please check comments
...nt/bio-commandline/src/main/kotlin/uk/ac/ebi/biostd/client/cli/services/SubmissionService.kt
Show resolved
Hide resolved
val client = bioWebClient(rqt.securityConfig) | ||
return client.submitSingle(rqt.submissionFile, rqt.filesConfig).body | ||
val client = bioWebClient(request.securityConfig) | ||
val acceptedSubmission = client.asyncSubmitSingle(request.submissionFile, request.filesConfig) |
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.
val (accNo, version) = client.asyncSubmitSingle(request.submissionFile, request.filesConfig)
client/bio-commandline/src/main/kotlin/uk/ac/ebi/biostd/client/cli/commands/SubmitCommand.kt
Outdated
Show resolved
Hide resolved
...ebapp/src/main/kotlin/ac/uk/ebi/biostd/submission/web/resources/SubmissionRequestResource.kt
Show resolved
Hide resolved
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.
Lookin good please check comments
- New wait operation that waits indefinitely - Add flag to wait indefinitely - Update imports
- Fix typo - Remove unnecessary annotations
https://www.pivotaltracker.com/story/show/187734050