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

Version query parameter not working for Dockstore/TRS import #12017

Closed
coverbeck opened this issue May 19, 2021 · 7 comments
Closed

Version query parameter not working for Dockstore/TRS import #12017

coverbeck opened this issue May 19, 2021 · 7 comments
Assignees
Labels

Comments

@coverbeck
Copy link

Dockstore is implementing a Launch with Galaxy button. When we specify the version, using the trs_version query parameter, the Galaxy UI shows an error, and you are prompted which version to select. The expected behavior is that Galaxy should import the specified version.

To reproduce, go to https://usegalaxy.org/workflows/trs_import?trs_server=dockstore.org&trs_id=%23workflow/github.com/jmchilton/galaxy-workflow-dockstore-example-1/mycoolworkflow&trs_version=master

Screen Shot 2021-05-19 at 10 17 18 AM

Also see discussion on Dockstore issue.

Tangential Issue

Let me know if I should log this separately. We'd like to specify a different Dockstore server for testing purposes. We have our daily build and staging environments. When we specify ...?trs_server=dev.dockstore.net&..., an error pops up briefly and Galaxy seems to read from dockstore.org. Is this supposed to work?

@mvdbeek mvdbeek self-assigned this May 19, 2021
@mvdbeek
Copy link
Member

mvdbeek commented May 19, 2021

When we specify ...?trs_server=dev.dockstore.net&..., an error pops up briefly and Galaxy seems to read from dockstore.org. Is this supposed to work?

It is, but we need to add dev.dockstore.net to the enabled and validated trs server list, I will make sure we enable this for usegalaxy.org / test.galaxyproject.org.

The version handling seems slightly off, I'll have a fix shortly.

@coverbeck
Copy link
Author

It is, but we need to add dev.dockstore.net to the enabled and validated trs server list, I will make sure we enable this for usegalaxy.org / test.galaxyproject.org.

Great! While you're doing that, could you please add our staging environment as well, staging.dockstore.org? So 3 total:

  • dockstore.org
  • staging.dockstore.org
  • dev.dockstore.net

Thanks!

@mvdbeek
Copy link
Member

mvdbeek commented May 19, 2021

OK, so this is a discrepancy between what workflowhub.eu defines as the "trs_version". For workflowhub.eu, this is what matches the id field, while for dockstore.org this matches the name. The name field looks better to me, since it actually contains the user-defined version of the workflow, but it would be great if this can be synced up (@bedroesb @bgruening you have workflowhub connections, right ?).

This is the workflowhub.eu:
Screenshot 2021-05-19 at 21 49 10
This is dockstore.org:
Screenshot 2021-05-19 at 21 51 49

I'm just gonna allow both for now, but this should really be resolved.

@bgruening
Copy link
Member

This is just poorly defined in the spec and its not clear for the workfowhub people. @coverbeck this is the related issue: ga4gh/tool-registry-service-schemas#153

On usegalaxy.eu we ship actually a patch to support both version.

@coverbeck
Copy link
Author

Ah, right, thanks, I remember that ticket now @bgruening.

@bgruening
Copy link
Member

Your welcome! Thanks for looking into it!

@mvdbeek
Copy link
Member

mvdbeek commented Jun 15, 2021

Alright, #12021 fixed the query parameter, and I've enabled dev and staging servers on https://test.galaxyproject.org/. Thanks for the report @coverbeck!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants