Make sure that the TMPDIR is used as STORAGE if it is set#911
Make sure that the TMPDIR is used as STORAGE if it is set#911trz42 merged 2 commits intoEESSI:2023.06-software.eessi.iofrom casparvl:if_set_use_tmpdir
Conversation
|
Instance
|
|
Instance
|
|
Instance
|
|
Instance
|
|
Instance
|
|
Instance
|
trz42
left a comment
There was a problem hiding this comment.
We should limit this (bind-mounting $TMPDIR) to cases where it is needed.
The setting of $STORAGE in bot/build.sh is ok, the bind-mount part should be moved to eessi_container.sh
…e TMP_IN_CONTAINER doesn't already contain that prefix
|
Ok, so that worked #912 (comment) , I'm happy with this. I checked using that there weren't any |
|
PR merged! Moved |
|
PR merged! Moved |
|
PR merged! Moved |
|
PR merged! Moved |
This is one of three pieces split off from #903
Together with EESSI/eessi-bot-software-layer#299 this makes sure that
TMPDIRis set tolocal_tmpas configured inapp.cfg, and then with this PR we ensure that thatTMPDIRis actually used everywhere if it is set. We also bind-mount it in the container, so that if any subsequent script calls amktempin the container, that is able to just created a dir within the mounted TMPDIR.