Skip to content

Fix apptainer and singularity --no-https option#6216

Merged
pditommaso merged 4 commits intonextflow-io:masterfrom
steven-pilger:patch-1
Jun 27, 2025
Merged

Fix apptainer and singularity --no-https option#6216
pditommaso merged 4 commits intonextflow-io:masterfrom
steven-pilger:patch-1

Conversation

@steven-pilger
Copy link
Contributor

@steven-pilger steven-pilger commented Jun 25, 2025

Fixes: #4591

Signed-off-by: Steven Pilger <8706215+steven-pilger@users.noreply.github.com>
Signed-off-by: Steven Pilger <8706215+steven-pilger@users.noreply.github.com>
Signed-off-by: Steven Pilger <8706215+steven-pilger@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Jun 25, 2025

Deploy Preview for nextflow-docs-staging ready!

Name Link
🔨 Latest commit dbe8baf
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/685e4ad99584d400083e9838
😎 Deploy Preview https://deploy-preview-6216--nextflow-docs-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bentsherman
Copy link
Member

Thanks for contributing. Can you figure out which versions of Singularity / Apptainer renamed this option? If we merge this, we'll need to document that as a minimum requirement

@steven-pilger
Copy link
Contributor Author

Looking into it 👀

Seems like the last entry in the changelog of singularity already talks about deprecating --nohttps:
https://github.com/apptainer/singularity/blob/9dceb4240c12b4cff1da94630d422a3422b39fcf/CHANGELOG.md?plain=1#L55
So that would be singularity version 3.8.4, while 'current' version is 3.8.7 from March 2022.

This is the same for apptainer as it was renamed around that time if I see that right. I can not find the commit that finally drops the support for --nohttps. Is this sufficient? Should this be documented somewhere?

@pditommaso pditommaso merged commit 66ead2c into nextflow-io:master Jun 27, 2025
5 checks passed
@pditommaso
Copy link
Member

Thanks for contributing this patch

@pditommaso
Copy link
Member

Oops, didn't notice @bentsherman comment

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update 'noHttps' option for both Singularity and Apptainer

3 participants