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

api: add endpoint for workflow status #46

Merged
merged 3 commits into from
Nov 7, 2017

Conversation

diegodelemos
Copy link
Member

@diegodelemos diegodelemos commented Nov 3, 2017

hjhsalo and others added 2 commits November 6, 2017 16:11
* add endpoint for getting status of a workflow
  (addresses reanahub/reana-client#39)

Signed-off-by: Harri Hirvonsalo <harri.hirvonsalo@cern.ch>
Signed-off-by: Diego Rodriguez <diego.rodriguez@cern.ch>
@diegodelemos diegodelemos force-pushed the 39-add-status-command branch 3 times, most recently from eac80a7 to b6679d5 Compare November 6, 2017 16:26
* Refactors REST endpoint for getting workflow status (`/api/workflows/:uuid/status`) to simplify
  the code logic and to speed up command execution.

* Adds additional tests for a situation with unknown workflow and for an unauthorised access.

Signed-off-by: Diego Rodriguez <diego.rodriguez@cern.ch>
@tiborsimko tiborsimko merged commit ff82e7b into reanahub:master Nov 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

cli: add status command
3 participants