Skip to content

enhance Rust easyblock to allow specifying channel to use (stable vs nightly)#3749

Merged
Micket merged 1 commit intoeasybuilders:developfrom
boegel:20250531114746_new_pr_rust
Jun 2, 2025
Merged

enhance Rust easyblock to allow specifying channel to use (stable vs nightly)#3749
Micket merged 1 commit intoeasybuilders:developfrom
boegel:20250531114746_new_pr_rust

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented May 31, 2025

(created using eb --new-pr)

@boegel
Copy link
Copy Markdown
Member Author

boegel commented May 31, 2025

@boegelbot please test @ jsc-zen3
EB_ARGS="Rust-1.52.1-GCCcore-10.3.0.eb Rust-1.70.0-GCCcore-12.3.0.eb Rust-1.85.1-GCCcore-14.2.0.eb"
CORE_CNT=16

@boegelbot
Copy link
Copy Markdown

@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=3749 EB_ARGS="Rust-1.52.1-GCCcore-10.3.0.eb Rust-1.70.0-GCCcore-12.3.0.eb Rust-1.85.1-GCCcore-14.2.0.eb" EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3749 --ntasks="16" ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 6544

Test results coming soon (I hope)...

Details

- notification for comment with ID 2925385730 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown

Test report by @boegelbot

Overview of tested easyconfigs (in order)

  • SUCCESS Rust-1.52.1-GCCcore-10.3.0.eb
  • SUCCESS Rust-1.70.0-GCCcore-12.3.0.eb
  • SUCCESS Rust-1.85.1-GCCcore-14.2.0.eb

Build succeeded for 3 out of 3 (3 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/a738ef5826fe3885818334e4e0ef782d for a full test report.

Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

since we are going to have multiple versions of this now, can be please consider dropping the html docs?

They take up 700MB out of 1.6GB and of course not a single person is going to read those on a cluster (you'd have to have a browser on the cluster, and then also somehow know to navigate into the Rust installation directory share. No one would ever prefer this to just reading it online).

--disable-docs
--disable-compiler-docs

Not sure if both or just one of these are needed

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jun 2, 2025

since we are going to have multiple versions of this now, can be please consider dropping the html docs?

They take up 700MB out of 1.6GB and of course not a single person is going to read those on a cluster (you'd have to have a browser on the cluster, and then also somehow know to navigate into the Rust installation directory share. No one would ever prefer this to just reading it online).

--disable-docs
--disable-compiler-docs

Not sure if both or just one of these are needed

I'm happy to, but let's do that in a follow-up PR since it has nothing to do with the changes being made here?

Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

ok then

@Micket Micket merged commit 59cc543 into easybuilders:develop Jun 2, 2025
17 checks passed
@boegel boegel deleted the 20250531114746_new_pr_rust branch June 3, 2025 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants