Skip to content

legion: disable parallel build for +rocm - #6121

Merged
rbberger merged 2 commits into
spack:developfrom
scheibelp:legion-rocm-disable-parallel
Aug 19, 2026
Merged

legion: disable parallel build for +rocm#6121
rbberger merged 2 commits into
spack:developfrom
scheibelp:legion-rocm-disable-parallel

Conversation

@scheibelp

Copy link
Copy Markdown
Member

Extract legion changes for rocm build from #4411

I noticed a problem with an intermittent build error (some sort of race condition) in CI when building legion+rocm in #4411

@spackbot-triage spackbot-triage Bot added the update-package Modifications to packages in the repository label Aug 17, 2026
@scheibelp scheibelp mentioned this pull request Aug 17, 2026
2 tasks
@rbberger

Copy link
Copy Markdown
Member

This should be revisited with the upcoming Legion/Realm September releases.

@elliottslaughter

Copy link
Copy Markdown
Contributor

Legion master has already merged the fix. Can we version bound this to 26.06 and earlier?

@scheibelp

Copy link
Copy Markdown
Member Author

Legion master has already merged the fix. Can we version bound this to 26.06 and earlier?

I've updated the constraint so that it only disables parallel builds for @:26.06 +rocm

bernhardkaindl
bernhardkaindl previously approved these changes Aug 18, 2026

@bernhardkaindl bernhardkaindl 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.

PS: I see legion is used in CI, but I don't see that +rocm is enabled, so I think it should not change CI duration:

git grep '\<legion\>' .ci/
.ci/gitlab/configs/linux/ci.yaml:      - legion
grep rocm .ci/gitlab/configs/linux/ci.yaml

@bernhardkaindl bernhardkaindl 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.

@alalazo I un-pushed my accidential push, but now it needs another approver to merge.

@rbberger

Copy link
Copy Markdown
Member

@spackbot run pipeline

@spackbot-app

spackbot-app Bot commented Aug 19, 2026

Copy link
Copy Markdown

I've started that pipeline for you!

@rbberger rbberger changed the title Disable parallel build for legion+rocm legion: disable parallel build for +rocm Aug 19, 2026
@spackbot-triage
spackbot-triage Bot requested a review from rbberger August 19, 2026 03:26
@rbberger
rbberger enabled auto-merge (squash) August 19, 2026 03:27
@rbberger
rbberger merged commit c61cfa7 into spack:develop Aug 19, 2026
65 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

update-package Modifications to packages in the repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants