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
api: add initialize workflow endpoint #40
api: add initialize workflow endpoint #40
Conversation
diegodelemos
commented
Oct 20, 2017
•
edited
edited
- (Closes workflow uuid should be created by reana-wotrkflow-controller reana-workflow-engine-yadage#32)
f53ff3a
to
f908f28
Compare
docs/openapi.json
Outdated
@@ -157,8 +206,7 @@ | |||
"description": "Request succeeded. The file has been added to the workspace.", | |||
"examples": { | |||
"application/json": { | |||
"message": "Workflow has been added to the workspace", | |||
"workflow_id": "cdcf48b1-c2f3-4693-8230-b066e088c6ac" | |||
"message": "The file input.txt has been successfully transferred." |
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.
Better input.csv
or input.dat
instead of .txt
... but we can use some consistent examples across REANA, see mydata.csv
, mycode.py
etc used in musings elsewhere.
70d2ab3
to
a95c272
Compare
d01eb91
to
b6968c9
Compare
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.
LGTM
application/json: | ||
{ | ||
"message": "Workflow workspace has been created.", | ||
"workflow_id": "cdcf48b1-c2f3-4693-8230-b066e088c6ac" |
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.
Will this payload include a name of the workflow in the future?
b6968c9
to
64d0519
Compare
Signed-off-by: Diego Rodriguez <diego.rodriguez@cern.ch>
64d0519
to
93eaf55
Compare