Skip to content
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

Disable CUDA installation for RISC-V build hosts in bot/build.sh (and some other minor enhancements) #836

Merged

Conversation

bedroge
Copy link
Collaborator

@bedroge bedroge commented Dec 6, 2024

This PR

  • adds some extra variables for the repository id, name, and version, and uses them throughout the script
  • changes the check for the dev repo, as it basically checked if the repo id defined in the bot config equals "dev.eessi.io", but the repo id could in principle be anything. It's better to check against the real repository name.
  • puts $SINGULARITY_TMPDIR under $JOB_STORAGE (job's temporary directory) instead of under $PWD (shared job dir), as it will only contain temporary files created by Singularity. This solves an issue that I had on the RISC-V cluster (it couldn't remove some files from the home directory at the end of the container session), and I think it's better to do it like this anyway.

@bedroge bedroge added the enhancement New feature or request label Dec 6, 2024
@riscv-eessi-io-bot
Copy link

Instance eessi-bot-riscv is configured to build for:

  • architectures: riscv64/generic
  • repositories: riscv.eessi.io-20240402

Copy link

eessi-bot bot commented Dec 6, 2024

Instance eessi-bot-mc-aws is configured to build for:

  • architectures: x86_64/generic, x86_64/intel/haswell, x86_64/intel/skylake_avx512, x86_64/amd/zen2, x86_64/amd/zen3, aarch64/generic, aarch64/neoverse_n1, aarch64/neoverse_v1
  • repositories: eessi.io-2023.06-compat, eessi-hpc.org-2023.06-software, eessi-hpc.org-2023.06-compat, eessi.io-2023.06-software

Copy link

eessi-bot bot commented Dec 6, 2024

Instance eessi-bot-mc-azure is configured to build for:

  • architectures: x86_64/amd/zen4
  • repositories: eessi.io-2023.06-compat, eessi.io-2023.06-software

Copy link
Collaborator

@trz42 trz42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@trz42 trz42 merged commit 42abeb7 into EESSI:2023.06-software.eessi.io Dec 7, 2024
50 checks passed
Copy link

eessi-bot bot commented Dec 7, 2024

PR merged! Moved [] to /project/def-users/SHARED/trash_bin/EESSI/software-layer/2024.12.07

@riscv-eessi-io-bot
Copy link

PR merged! Moved [] to /home/eessibot/shared/trash_bin/EESSI/software-layer/2024.12.07

Copy link

eessi-bot bot commented Dec 7, 2024

PR merged! Moved [] to /project/def-users/SHARED/trash_bin/EESSI/software-layer/2024.12.07

@bedroge bedroge deleted the bot_build_enhancements_riscv branch December 7, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants