Skip to content

Fix redundant containers#11276

Open
SPPearce wants to merge 2 commits intomasterfrom
adjust_repeated_containers
Open

Fix redundant containers#11276
SPPearce wants to merge 2 commits intomasterfrom
adjust_repeated_containers

Conversation

@SPPearce
Copy link
Copy Markdown
Contributor

A couple of containers which had the same value for docker and singularity/apptainer.

conda "${moduleDir}/environment.yml"
container "${ workflow.containerEngine in ['singularity', 'apptainer'] && !task.ext.singularity_pull_docker_container ?
'community.wave.seqera.io/library/hmmer_prodigal_pandas_parallel_pruned:ccae2eabc2a54ac8' :
'oras://community.wave.seqera.io/library/hmmer_prodigal_pandas_parallel_pruned:9b929a08d061b63d' :
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be swapped to https?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't figure out how to get that from the wave-cli :|
We have a lot of oras links in the repository already.

conda "${moduleDir}/environment.yml"
container "${ workflow.containerEngine in ['singularity', 'apptainer'] && !task.ext.singularity_pull_docker_container ?
'community.wave.seqera.io/library/hmmer_prodigal_pandas_parallel_pruned:ccae2eabc2a54ac8' :
'oras://community.wave.seqera.io/library/hmmer_prodigal_pandas_parallel_pruned:9b929a08d061b63d' :
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants