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

STDOUT/STDERR logs to separate endpoints #127

Open
uniqueg opened this issue Nov 1, 2019 · 1 comment
Open

STDOUT/STDERR logs to separate endpoints #127

uniqueg opened this issue Nov 1, 2019 · 1 comment
Assignees
Labels
priority: high High priority status: in progress Currently being worked on type: feature New feature or request type flag: required by specs Compliance with specs requires this change type: maintenance Related to general repository maintenance workload: days Likely takes days to resolve

Comments

@uniqueg
Copy link
Member

uniqueg commented Nov 1, 2019

Is your feature request related to a problem? Please describe.
According to the WES v1.0.0 specs, stdout and stderr in the run and task logs of the GET /runs/{run_id} response should consist of URLs pointing to these logs, not the logs themselves (as is currently implemented).

Describe the solution you'd like
Add an OpenAPI spec for additional endpoints, e.g., /logs/stdout/{id} and /logs/stderr/{id} where run and task logs can be retrieved from via GET requests. Business logic for the existing GET /runs/{run_id} endpoint needs to be adjusted accordingly.

Describe alternatives you've considered
N/A

Additional context
N/A

@uniqueg uniqueg added priority: high High priority status: in progress Currently being worked on type flag: required by specs Compliance with specs requires this change type: feature New feature or request type: maintenance Related to general repository maintenance workload: days Likely takes days to resolve labels Apr 20, 2020
@uniqueg uniqueg self-assigned this Apr 20, 2020
@uniqueg
Copy link
Member Author

uniqueg commented Apr 20, 2020

Started implementation together with #128

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high High priority status: in progress Currently being worked on type: feature New feature or request type flag: required by specs Compliance with specs requires this change type: maintenance Related to general repository maintenance workload: days Likely takes days to resolve
Projects
None yet
Development

No branches or pull requests

1 participant