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

Update rsync data manager path modifications #2839

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

@davebx
Copy link
Contributor

@davebx davebx commented Jan 29, 2020

FOR CONTRIBUTOR:

  • - I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • - License permits unrestricted use (educational + commercial)
  • - This PR updates an existing tool or tool collection

As it stands, the location files specify the cvmfs path, so the data manager explodes when trying to match that against /galaxy/data/

This will resolve a question posted on galaxy help.

Additionally, the files in rsync://datacache.g2.bx.psu.edu/location were specifying paths under rsync://datacache.g2.bx.psu.edu/byhand, which did not match the code's indexes/ prefix, so it was attempting to fetch data from rsync://datacache.g2.bx.psu.edu/byhand/indexes.

As it stands, the location files specify the cvmfs path, so the data manager explodes when trying to match that against /galaxy/data/
@davebx davebx added the bug label Jan 29, 2020
@davebx davebx changed the title Update data path prefix [WIP] Update data path prefix Jan 29, 2020
@davebx davebx added the wip label Jan 29, 2020
@jennaj
Copy link
Member

@jennaj jennaj commented Feb 21, 2020

Any updates? Looks like v 0.0.3 is still the most current https://toolshed.g2.bx.psu.edu/view/devteam/data_manager_rsync_g2/e0329ab30f6d

ping @davebx @natefoo

Loading

@davebx davebx added reviewer needed and removed wip labels Mar 4, 2020
@davebx davebx changed the title [WIP] Update data path prefix Update rsync data manager path modifications Mar 4, 2020
@davebx
Copy link
Contributor Author

@davebx davebx commented Mar 5, 2020

The path update broke the test, WIP again until I fix it.

Loading

@davebx davebx added wip and removed reviewer needed labels Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants