-
Notifications
You must be signed in to change notification settings - Fork 53
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
remove /service-info endpoint (for now) #283
remove /service-info endpoint (for now) #283
Conversation
Per PRC discussion, the GA4GH-wide /service-info spec isn't yet ready for use.
I don't think this endpoint needs deletion for DRS v1, as points raised need to be addressed by Discovery and future inclusion post v1 |
I'm personally fine either way, so unless the PRC has a strong opinion, I defer to the group. The best argument for deleting it now is that, once there's a GA4GH-blessed cross-API |
I'd keep the Once we get a GA4GH blessed |
+0 ELIXIR |
+1 Terra |
+1 from DCF/UChicago/CTDS/Gen3/ |
+0 from Seven Bridges (CGC, Cavatica) |
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.
Farewell /service-info, I'm sure we will see you again soon!
Seriously, though, I think it's best to remove it for now and I suspect it will be added in very late in the game before Oct's meeting. Implementers, I think it's fine to keep this in your implementations and tweak when the final version comes out.
Addresses DRS v1 PRC #11 /service-info items
Per PRC discussion, the GA4GH-wide /service-info spec isn't yet ready for use.