Implementing GA4GH service-info for TES

ga4gh / task-execution-schemas

Apache License 2.0

80 stars 28 forks source link

Implementing GA4GH service-info for TES #129

Closed susheel closed 3 years ago

susheel commented 4 years ago

Following instructions https://github.com/ga4gh-discovery/ga4gh-service-info#how-to-use-and-extend-this-specification

This introduces breaking changes:

d432788 /tasks/service-info does not exist
6b3af5d /service-info new endpoint
[ ] Bump OAPI from 2 to 3
[ ] Merge storage array dict with GA4GH service-info spec

Resolves #128

susheel commented 4 years ago

Rebased this to release/v0.4.1

susheel commented 4 years ago

@delagoya Noted. I have added these as tasks in the description. If we need to merge the dict, we would also probably need to upgrade to OAPI3 as allOf is not supported in OAPI2 AFAIK

susheel commented 4 years ago

GA4GH service-info has just tagged their release v1.0.0 which is now updated in the PR.

Merge with existing TES dict needs planning

aniewielska commented 3 years ago

GA4GH service-info has been in the end implemented in TES in #132 together with OpenAPI 3 migration and changed further in #143. Thanks for starting this effort here.