Skip to content

roslin.config: Few fixes#892

Merged
sguizard merged 1 commit intonf-core:masterfrom
sguizard:patch-01_cache_logic_remove_igenome_fix_singularity_version
Apr 21, 2025
Merged

roslin.config: Few fixes#892
sguizard merged 1 commit intonf-core:masterfrom
sguizard:patch-01_cache_logic_remove_igenome_fix_singularity_version

Conversation

@sguizard
Copy link
Copy Markdown
Contributor


name: roslin.config
about: Few fixes for smooth runs

  • Remove the igenome configuration. IGMM igenome folder is not accessible for Roslin users.
  • Load singularity 3 instead of 4. Version 4 throw squashfs / FUSE error on new nodes.
  • Add logic to define the singularity cache directory. Users who don't use compute project can't access the shared singularity cache.

@sguizard sguizard changed the title conf/roslin.config: update singularity cache directory handling, remove igenome directory, force singularity3 roslin.config: Few fixes Apr 21, 2025
Comment thread conf/roslin.config
// Define the singularity cache directory depending on the presence of the NFX_SGE_PROJECT variable
// User without compute project can't access to the shared cache directory.
// So, they need to store singularity images into the work directory.
cacheDir = System.getenv('NFX_SGE_PROJECT') ?
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

libraryDir instead? (In which case one might not need the conditional?)

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.

The cacheDir better fit to our setup. It convenient for us to allow user to add new images when they run a new pipeline / new version.
Thanks for the suggestion.

Comment thread conf/roslin.config
"""
. /etc/profile.d/modules.sh
module load singularity
module load igmm/apps/singularity/3
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'd strongly recommend trying to resolve this, the last 3 release is almost two years old and there's been a steady stream of fixes since.

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.

We have updated our nodes very recently.
The IS are working on this issue.
While waiting for a resolution, non nextflow power user struggle to debug this singularity.
I'll switch back as soon as singularity 4 is fixed ;)

@sguizard
Copy link
Copy Markdown
Contributor Author

Many thanks for the review and the suggestions!

@sguizard sguizard merged commit 17e88dd into nf-core:master Apr 21, 2025
148 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants