-
Notifications
You must be signed in to change notification settings - Fork 2
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 TMPDIR environment variables to handle non-default temp directories #12
Conversation
thank you for sending that pull request. I am hoping to be able to test this next week. Did you test your changes on your system and did it work? I think it also needs mount points in addition for this to work? |
Yes, you are right. I copied over what I had on our HPC, but indeed for us the non-default tmpdir is automatically mounted on all apptainers. So perhaps it is better to script this and check if any of the TMPDIR variables (as used in this PR) are non-empty and then bindmount them as |
So something like: CUSTOMTMPDIR="/tmp"
for CUSTOMDIR in $TMP $TMPDIR $TEMP $TEMPDIR; do
CUSTOMTMPDIR=$CUSTOMDIR
done
apptainer run --cleanenv --no-mount tmp --bind ${CUSTOMTMPDIR}:/tmp etc |
…riables (Github issue NeuroDesk#11)
cool! Does this work on your system now? Then I test on the HPCs I have access to and we can make it available for all |
If I manually edit our singularity run-scripts likewise then it works for us :-) |
thank you for confirming :) |
I didn't look into setting this bind option centrally, perhaps that is also possible as an alternative |
yes, that would also be possible |
Indeed, I now see that there exists a variable for this, i.e. |
I just tested the new run_fetch code and it works nicely :-) |
See Github issue #11