Skip to content

Conversation

@wdconinc
Copy link
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
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
Copy link

Copilot AI left a comment

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.

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>
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