-
Notifications
You must be signed in to change notification settings - Fork 72
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
Copy or move the files instead of creating symlinks in the LocalizeReads task for compatibility with CoA/TES. #619
Conversation
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.
Thanks @VJalili! Did you happen to test using cp
as well? We've had complaints in the past about moving mv
on input files from folks using shared filesystems. I realize it's slower and would require extra disk, but I would think that the cost would be small.
That is a good point, thank you, @mwalker174. Yes, I first tested using |
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.
Looks good! Minor comments
Co-authored-by: Mark Walker <[email protected]>
Thank you, @mwalker174!! |
The task
LocalizeReads
creates symlinks for the localized cram and its corresponding index file in the current working directory to simplify referencing them in the task output. Specifically, the files are localized in the following directory, and symlinks are created in the/cromwell_root/.
Symlinks as this are incompatible with GA4GH TES and Cromwell on Azure.
Therefore, this PR updates the script to move or copy the files instead of creating symlinks. Moving files (
mv
) is faster and requires less disk space on the VM than copying the files. However, run on HPC, moving files could lead to moving them from their source directory compared to moving them from the localized directory to the root directory on the VM. Therefore, we provide both options for moving and copying to avoid moving files unexpectedly and set the copy as default.Cromwell submission ID for a successful test of this PR is
bd10381c-53b3-4c67-a396-291135af0f6f
.The following can be an alternative to moving the files, though it still needs to be tested.