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.

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

cli: introducing run command #198

Merged
merged 1 commit into from
Nov 8, 2018

Conversation

roksys
Copy link
Contributor

@roksys roksys commented Nov 2, 2018

Signed-off-by: Rokas Maciulaitis rokas.maciulaitis@cern.ch

@ghost ghost assigned roksys Nov 2, 2018
@ghost ghost added the Status: in review label Nov 2, 2018
@roksys roksys force-pushed the run-wrapper-command branch 3 times, most recently from 0d6550a to 8910e8c Compare November 6, 2018 16:12
@roksys roksys removed the Need: work label Nov 6, 2018
@tiborsimko tiborsimko added this to the Live-Mode milestone Nov 7, 2018
tests/conftest.py Outdated Show resolved Hide resolved
@tiborsimko
Copy link
Member

Works well, I left a few cosmetic comments.

Please amend also docs/userguide so that people know how about this wrapper command.

You can say that if people have fully working example with reana.yaml, they can run the workflow easily via the run wrapper command, which will create a new workflow run, upload analysis inputs, and start the workflow run. You can present a small usage scenario going like:

$ vim reana.yaml
$  reana-client run -n myanalysis
$ export REANA_WORKON=myanalysis
$ # ... after a minute...
$ reana-client status 
$ reana-client download results/plot.png

so that people see the full circle.

docs/userguide.rst Outdated Show resolved Hide resolved
* introducing run command, which wrapps `create`, `upload` & `start`.
  Closes reanahub#196

* deleting unused imports.

Signed-off-by: Rokas Maciulaitis <rokas.maciulaitis@cern.ch>
@tiborsimko tiborsimko merged commit d5dc320 into reanahub:master Nov 8, 2018
@ghost ghost removed the Status: in review label Nov 8, 2018
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