introduce variable $EESSI_INIT_PREFIX for overriding location of init directory
#93
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen2 |
|
New job on instance
|
|
New job on instance
|
bedroge
added
ready-to-deploy
2025.06-software.eessi.io
Contributor
Author
|
Removed the ready-to-deploy label, need to do a similar change to the bash init scripts (that's what the CI is complaining about). |
Contributor
Author
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen2 |
|
New job on instance
|
|
New job on instance
|
Contributor
Author
|
Tested the additional changes in the init scripts with the EESSI container on a RISC-V system: 2023.06: 2025.06: 2025.06-1: |
boegel
reviewed
Oct 21, 2025
| echo "" | ||
| fi | ||
| elif [[ "${EESSI_VERSION_DEFAULT}" == "2025.06" ]]; then | ||
| export EESSI_CVMFS_REPO="${EESSI_CVMFS_REPO_OVERRIDE:=/cvmfs/dev.eessi.io/riscv}" |
Contributor
There was a problem hiding this comment.
@bedroge Can you make sure that https://www.eessi.io/docs/repositories/riscv.eessi.io/ gets updated accordingly?
Contributor
|
@bedroge |
Contributor
|
staging PR merged for We should wait to trigger those builds until updated |
Contributor
Author
Oh, yes, you're absolutely right, totally overlooked that! |
Contributor
Author
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen3 |
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
|
New job on instance
|
Contributor
|
2nd staging PR merged, so |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This doesn't change any default behavior, but it's needed for the RISC-V bot: RISC-V software is built for
/cvmfs/dev.eessi.io/riscv, while using the compat layer and scripts fromsoftware.eessi.io.$EESSI_PREFIXis a bit ambiguous in that case, and is currently pointing to/cvmfs/dev.eessi.io/riscv, meaning that the EESSI-extend module will also look there for the init dir (that's where it expects the EB hooks file). This new variable allows us to to override that location.These two changes were tested in EESSI/dev.eessi.io-riscv#1 (comment), along with some additional changes in https://github.com/EESSI/dev.eessi.io-scripts/blob/main/bot/bot-build-dev.eessi.io.slurm (this needs to actually set that variable).
edit: though not really required, I figured it would be clean to have the EESSI module set
EESSI_INIT_PREFIXas well, so I've added that in 1ff11e3. For consistency, I also renamed the local variableeessi_archdetect_prefixtoeessi_init_prefix.