-
Notifications
You must be signed in to change notification settings - Fork 16
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
add apptainer
explicitly to the ldmx-env.sh
script
#1248
Comments
Keeping track of versions floating around for compatibility. Many of these clusters have auto-updating enabled and so the version numbers may change. For our purposes, I'm mainly interested in which flavour1 of singularity is being used since that is the largest compatibility hurdle we currently face.
Footnotes
|
It looks like many of these clusters use module as a means for gaining access to apptainer. This could be a QoL feature for the ldmx-env.sh script to attempt |
Is your feature request related to a problem? Please describe.
@jpasc27 informed me that UVA has updated from singularity to apptainer on their cluster recently. This led to a discussion about including apptainer explicitly in the environment script so that folks don't accidentally use the old version of singularity.
apptainer
installs with a symlink namedsingularity
which has prevented this from being an issue in the past. For example, at UMN:I've already done this within
denv
#1232 so it should be pretty easy for me to do this here. I'm just putting notes here for now because I'm not sure if all of our institutions have moved to apptainer yet.Describe the solution you'd like
This update is only necessary if both a relic singularity and a new apptainer are installed at once.
Here are the locations in the ldmx-env.sh script that (I think) would need to be updated (untested)
ldmx-sw/scripts/ldmx-env.sh
Lines 34 to 42 in a0a28a3
include an
apptainer
elif
check beforesingularity
ldmx-sw/scripts/ldmx-env.sh
Line 170 in a0a28a3
have this be
apptainer
ORsingularity
because their CLI are so similar.ldmx-sw/scripts/ldmx-env.sh
Lines 183 to 185 in a0a28a3
apptainer
renamed this env var toAPPTAINER_CACHEDIR
, so to avoid a warning during some runs, we should change this name if the runner is apptainer.Describe alternatives you've considered
An alternative is to switch to
denv
#1232 but that is a bigger change that just adding another runner explicitly.The text was updated successfully, but these errors were encountered: