Skip to content

feat: support apptainer (in particular avoid warnings)#1

Open
wdconinc wants to merge 5 commits into
mainfrom
apptainer-support
Open

feat: support apptainer (in particular avoid warnings)#1
wdconinc wants to merge 5 commits into
mainfrom
apptainer-support

Conversation

@wdconinc

Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce?

Apptainer warns when SINGULARITY_* and APPTAINER_* variables are different. This will make the user experience smoother with fewer warnings.

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Documentation update
  • Other: __

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

No.

Does this PR change default behavior?

No.

@wdconinc wdconinc self-assigned this Jan 11, 2023
@wdconinc wdconinc requested a review from sly2j January 11, 2023 21:04
Comment thread install.sh Outdated
Comment thread install.sh
Comment thread install.sh Outdated
Copilot AI review requested due to automatic review settings January 18, 2026 23:35
@wdconinc wdconinc requested a review from veprbl January 18, 2026 23:36
@wdconinc wdconinc enabled auto-merge (squash) January 18, 2026 23:36

Copilot AI left a comment

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.

Pull request overview

This PR adds support for Apptainer to prevent warnings when SINGULARITY_* and APPTAINER_* environment variables differ. The changes prioritize Apptainer over Singularity when both are available, set corresponding APPTAINER_* environment variables alongside existing SINGULARITY_* variables, and update user-facing messages and documentation to reflect support for both tools.

Changes:

  • Added APPTAINER_TMPDIR and APPTAINER_BINDPATH environment variable exports
  • Modified binary detection logic to prefer Apptainer over Singularity
  • Updated error messages, help text, and comments to reference Apptainer alongside Singularity

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread install.sh Outdated
Comment thread install.sh
Comment thread install.sh Outdated
Comment thread install.sh Outdated
Comment thread install.sh
wdconinc and others added 4 commits January 18, 2026 17:59
Co-authored-by: Dmitry Kalinkin <dmitry.kalinkin@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@netlify

netlify Bot commented Apr 30, 2026

Copy link
Copy Markdown

Deploy Preview for kaleidoscopic-syrniki-0cfb8a failed.

Name Link
🔨 Latest commit 9db983d
🔍 Latest deploy log https://app.netlify.com/projects/kaleidoscopic-syrniki-0cfb8a/deploys/69f351865a766f00086e792e

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.

3 participants