-
Notifications
You must be signed in to change notification settings - Fork 50
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
Let 2023.06 modulefile redirect RISC-V clients to riscv.eessi.io, add symlink for 20240402 version #840
Let 2023.06 modulefile redirect RISC-V clients to riscv.eessi.io, add symlink for 20240402 version #840
Conversation
Instance
|
Instance
|
Instance
|
Successful build with the RISC-V bot using this approach in #803 (comment). The latter still had a hardcoded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
bot: build repo:eessi.io-2023.06-software arch:x86_64/generic |
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
Tarball got ingested. New module file is available via CernVM-FS. |
PR merged! Moved |
PR merged! Moved |
PR merged! Moved |
This allows RISC-V clients (and, in particular, the build bot) to use the same modulefile as we use for software.eessi.io. By adding a symlink
20240402.lua -> 2023.06.lua
we can reuse the exact same modulefile. Note that I haven't added the symlink toinstall_scripts.sh
, which ensures that it won't be deployed to any repo (it would be confusing to have this in software.eessi.io, and we don't actually need it in riscv.eessi.io either, as long as it's available in the git repo; the bot sources the file from the git repo). The only slightly confusing thing is that the RISC-V repo will deploy the 2023.06.lua modulefile, so users from that repo will only seeEESSI/2023.06
. But I'm not really sure how to prevent that, except by adding a bunch of specific if statements toinstall_scripts.sh
.Instead of this approach we could also modify the EESSI-install-software.sh script and do something like this:
But I liked the solution with the modulefile a bit more.