Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.0] Use apptainer for SingularityComputingElement and enhance debugging #8000

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

chrisburr
Copy link
Member

@chrisburr chrisburr commented Jan 21, 2025

BEGINRELEASENOTES

*WorkloadManagement
CHANGE: Use apptainer for SingularityComputingElement
CHANGE: Drop support for SingularityComputingElement without user namespaces
CHANGE: Enchance debugging output if SingularityComputingElement fails
CHANGE: Drop support for using apptainer from outside of DIRACOS2

ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Jan 21, 2025
@chrisburr chrisburr changed the title [master] Use apptainer for SingularityComputingElement and enhance debugging [8.0] Use apptainer for SingularityComputingElement and enhance debugging Jan 21, 2025
@chrisburr chrisburr force-pushed the apptainer-ce branch 2 times, most recently from b549e0b to 423b9ab Compare January 22, 2025 14:13
@chrisburr chrisburr marked this pull request as ready for review January 22, 2025 14:30
@chrisburr chrisburr force-pushed the apptainer-ce branch 2 times, most recently from ffb51c0 to b16c65b Compare January 22, 2025 14:31
@aldbr
Copy link
Contributor

aldbr commented Jan 22, 2025

Shouldn't we use apptainer instead of singularity here?

FALLBACK_SINGULARITY = "/cvmfs/oasis.opensciencegrid.org/mis/singularity/current/bin"

$ ls -l /cvmfs/oasis.opensciencegrid.org/mis/
total 228
drwxrwxr-x. 33 cvmfs cvmfs   4096 Dec 16 21:48 apptainer
drwxr-xr-x. 29 cvmfs cvmfs   4096 Apr  3  2024 singularity

@chrisburr
Copy link
Member Author

FALLBACK_SINGULARITY was now unused so I've removed it.

@chrisburr chrisburr merged commit 78d20af into DIRACGrid:rel-v8r0 Jan 27, 2025
26 checks passed
@DIRACGridBot DIRACGridBot added sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention labels Jan 27, 2025
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/12985893600

Failed:

  • integration
    cherry-pick 78d20af into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-78d20afae-integration
    git cherry-pick -x -m 1 78d20afae
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #8000 Use apptainer for SingularityComputingElement and enhance debugging' --author='Chris Burr <[email protected]>'
    git push -u origin cherry-pick-2-78d20afae-integration
    
    # If you have the GitHub CLI installed the PR can be made with
    gh pr create \
         --label 'sweep:from rel-v8r0' \
         --base integration \
         --repo DIRACGrid/DIRAC \
         --title '[sweep:integration] Use apptainer for SingularityComputingElement and enhance debugging' \
         --body 'Sweep #8000 `Use apptainer for SingularityComputingElement and enhance debugging` to `integration`.
    
    Adding original author @chrisburr as watcher.
    
    BEGINRELEASENOTES
    
    *WorkloadManagement
    CHANGE: Use apptainer for SingularityComputingElement
    CHANGE: Drop support for SingularityComputingElement without user namespaces
    CHANGE: Enchance debugging output if SingularityComputingElement fails
    CHANGE: Drop support for using apptainer from outside of DIRACOS2
    
    ENDRELEASENOTES
    Closes #8009'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants