Skip to content

update Mesa easyblock to remove osmesa part of the sanity check, since that's removed from Mesa 25.1 onwards#3896

Merged
boegel merged 2 commits intoeasybuilders:developfrom
casparvl:remove_osmesa_from_mesa_sanity
Aug 27, 2025
Merged

update Mesa easyblock to remove osmesa part of the sanity check, since that's removed from Mesa 25.1 onwards#3896
boegel merged 2 commits intoeasybuilders:developfrom
casparvl:remove_osmesa_from_mesa_sanity

Conversation

@casparvl
Copy link
Copy Markdown
Contributor

Untested for now, but as stated in easybuilders/easybuild-easyconfigs#23144 (comment) we should adapt the sanity check for Mesa 25.1 onwards, since that's when osmesa got removed (see https://docs.mesa3d.org/relnotes/25.1.0.html and https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33836 ).

Tested to work with Mesa 25.1.3 from easybuilders/easybuild-easyconfigs#23144 . Now also testing with older versions to confirm it didn't break anything.

@casparvl
Copy link
Copy Markdown
Contributor Author

casparvl commented Aug 23, 2025

@boegelbot please test @ jsc-zen3
EB_ARGS=" --from-pr 23144 Mesa-25.1.3-GCCcore-14.2.0.eb Mesa-24.1.3-GCCcore-13.3
.0.eb Mesa-23.1.9-GCCcore-13.2.0.eb"

@boegelbot
Copy link
Copy Markdown

@casparvl: 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=3896 EB_ARGS=" --from-pr 23144 Mesa-25.1.3-GCCcore-14.2.0.eb Mesa-24.1.3-GCCcore-13.3 .0.eb Mesa-23.1.9-GCCcore-13.2.0.eb" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3896 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3216546063 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).

@casparvl
Copy link
Copy Markdown
Contributor Author

@Crivella
Copy link
Copy Markdown
Contributor

@boegelbot please test @ jsc-zen3
EB_ARGS="Mesa-24.1.3-GCCcore-13.3.0.eb Mesa-23.1.9-GCCcore-13.2.0.eb"

@boegelbot
Copy link
Copy Markdown

@Crivella: 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=3896 EB_ARGS="Mesa-24.1.3-GCCcore-13.3.0.eb Mesa-23.1.9-GCCcore-13.2.0.eb" EB_CONTAINER= EB_REPO=easybuild-easyblocks EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_3896 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 3219664535 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 Mesa-24.1.3-GCCcore-13.3.0.eb
  • SUCCESS Mesa-23.1.9-GCCcore-13.2.0.eb

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

Copy link
Copy Markdown
Contributor

@Crivella Crivella left a comment

Choose a reason for hiding this comment

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

LGTM

Once we have the latest changes in easybuilders/easybuild-easyconfigs#23144 we can also test 25.1 and merge this

@Crivella Crivella added this to the next release (5.1.2) milestone Aug 25, 2025
@boegel boegel changed the title Remove osmesa part of the sanity check, since osmesa is removed from mesa 25.1 onwards update Mesa easyblock to remove osmesa part of the sanity check, since that's removed from Mesa 25.1 onwards Aug 27, 2025
@boegel boegel changed the title update Mesa easyblock to remove osmesa part of the sanity check, since that's removed from Mesa 25.1 onwards update Mesa easyblock to remove osmesa part of the sanity check, since that's removed from Mesa 25.1 onwards Aug 27, 2025
@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 27, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Mesa-22.0.3-GCCcore-11.3.0.eb
  • SUCCESS Mesa-23.1.4-GCCcore-12.3.0.eb
  • SUCCESS Mesa-23.1.9-GCCcore-13.2.0.eb
  • SUCCESS Mesa-24.1.3-GCCcore-13.3.0.eb

Build succeeded for 4 out of 4 (4 easyconfigs in total)
node3526.doduo.os - Linux RHEL 9.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.9.18
See https://gist.github.com/boegel/68672a0b3df75cf0df10f3f64208f1f4 for a full test report.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel merged commit d20aaf8 into easybuilders:develop Aug 27, 2025
17 checks passed
@boegel boegel added the update label Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants