Skip to content

do not run scipy easyblock tests for runtest = False - #4200

Merged
smoors merged 2 commits into
easybuilders:developfrom
Thyre:20260812124544_new_pr_scipy
Aug 12, 2026
Merged

do not run scipy easyblock tests for runtest = False#4200
smoors merged 2 commits into
easybuilders:developfrom
Thyre:20260812124544_new_pr_scipy

Conversation

@Thyre

@Thyre Thyre commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

(created using eb --new-pr)

Currently, a set runtest = False in the EasyConfig is completely ignored by the EasyBlock, causing the tests to be executed either way. On our systems, this can lead to a significant oversubscription due to user limits, depending on where the software is built (e.g. login node) and which user builds it. As a consequence, the tests are running for ages, partially into build job timeouts.

At the same time, we do not want to skip tests altogether. The NumPy and SciPy test suites can be executed later on, so we prefer this approach.

As such, allow a user to explicitly disable running the tests.

@Thyre

Thyre commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

Test report by @Thyre

Overview of tested easyconfigs (in order)

  • SUCCESS cryptography-47.0.0-GCCcore-15.2.0.eb

  • SUCCESS dulwich-1.2.0-GCCcore-15.2.0.eb

  • SUCCESS rpds-py-0.30.0-GCCcore-15.2.0.eb

  • SUCCESS scikit-build-core-0.12.2-GCCcore-15.2.0.eb

  • SUCCESS tibs-0.5.7-GCCcore-15.2.0.eb

  • SUCCESS watchfiles-1.1.1-GCCcore-15.2.0.eb

  • SUCCESS poetry-2.3.4-GCCcore-15.2.0.eb

  • SUCCESS Python-bundle-PyPI-2026.04-GCCcore-15.2.0.eb

  • SUCCESS pybind11-3.0.4-GCCcore-15.2.0.eb

  • SUCCESS spin-0.18-GCCcore-15.2.0.eb

  • SUCCESS hypothesis-6.153.2-GCCcore-15.2.0.eb

  • SUCCESS meson-python-0.19.0-GCCcore-15.2.0.eb

  • SUCCESS SciPy-bundle-2026.05-gcccoreflexiblas-15.2.0-3.5.0.eb

Build succeeded for 13 out of 13 (total: 50 mins 18 secs) (13 easyconfigs in total)
jsfl02.jusuf - Linux Rocky Linux 9.8, x86_64, AMD EPYC 7742 64-Core Processor, Python 3.9.25
See https://gist.github.com/Thyre/98869f0d1476a647ed9ffac0b041b99b for a full test report.

@Thyre

Thyre commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

@boegelbot please test @ jsc-zen3
CORE_CNT=16
EB_ARGS="--installpath /tmp/$USER/ebpr-4200 SciPy-bundle-2026.05-gfbf-2026.1.eb SciPy-bundle-2022.05-foss-2022a.eb"

@boegelbot

Copy link
Copy Markdown

@Thyre: 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=4200 EB_ARGS="--installpath /tmp/$USER/ebpr-4200 SciPy-bundle-2026.05-gfbf-2026.1.eb SciPy-bundle-2022.05-foss-2022a.eb" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_4200 --ntasks="16" ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 5266688669 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 SciPy-bundle-2026.05-gfbf-2026.1.eb

  • SUCCESS SciPy-bundle-2022.05-foss-2022a.eb

Build succeeded for 2 out of 2 (total: 1 hour 30 mins 8 secs) (2 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.8, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.25
See https://gist.github.com/boegelbot/8ae28ffe642865e4049f174e008b7969 for a full test report.

@smoors smoors added this to the next release (5.4.1?) milestone Aug 12, 2026

@smoors smoors left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@smoors

smoors commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Going in, thanks @Thyre!

@smoors
smoors merged commit 12f550a into easybuilders:develop Aug 12, 2026
22 of 42 checks passed
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