-
Notifications
You must be signed in to change notification settings - Fork 664
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
Error staging directory from S3 as input #3110
Comments
Can you please include the full .nextflow.log file? |
|
What's the output of |
|
The error is caused by the use of a non-official Java distribution, likely installed via Conda. See here for details #2841 (comment) |
Yup switching to a different OpenJDK version fixed it. Thanks! |
@pditommaso at least for nf-core, the docs state that nextflow can be installed via conda (bioconda). Do you think that one should NOT use conda for install, which also means not using the nf-core devcontainer (https://github.com/nf-core/tools/blob/master/.devcontainer/devcontainer.json), since it installs nextflow via conda? |
Yeah, it should be avoided. Even better it should look into patching the Conda (Bioconda) recipe for Java. tagging @ewels |
Any tips of which specific java version should be used then? |
For example AdoptJDK or just use https://sdkman.io/ to install it |
So nothing that can be done within the constraints of bioconda/conda-forge? |
It must be updated the conda-forge recipe to use an official Java version |
I think as long as Here's the OpenJDK from
But the issue is when
|
The docs already talk about setting up the conda channels properly, but we should remove the The devtainer setup should be fine, as it's based on the GitPod image, and that configures the channels properly before installing Nextflow. |
I like your suggestion of dropping the As for the version pinning, we can do that. Only potential issue would be version conflicts, but that might actually be useful to prevent future conda related issues appearing here. I think if it becomes too common, we could add postlink check to warn users an unsupported version of java is installed and "here's how to fix it". This would check would happen every time an environment containing Nextflow is activated. Let me know what you want to do. |
nf-core docs updated in nf-core/website#1538 👍🏻 |
Bug report
Expected behavior and actual behavior
I'd like to be able to stage a directory from S3 as input to task, but am getting a java.lang.UnsupportedOperationException error when doing so. I'd expect this to work as it does locally, where the local path is symlinked into the working directory, and works like any other file.
Steps to reproduce the problem
This workflow takes a directory path as param
input_dir
, and lists it's contents in a file:Program output
When running with local path it succeeds, when running with s3 path, it fails
from .nextflow.log:
Environment
Additional context
Using this to stage a directory of BWA Indices so I don't have to specify the individual files as s3 inputs. Would like to download them from s3 so can run alignment on AWS Batch.
The text was updated successfully, but these errors were encountered: