Skip to content

[workflows] Run extended ROCgdb tests in parallel on PRs - #245

Draft
lumachad wants to merge 1 commit into
amd-stagingfrom
users/lumachad/amd-staging/extended-testing-v2
Draft

[workflows] Run extended ROCgdb tests in parallel on PRs#245
lumachad wants to merge 1 commit into
amd-stagingfrom
users/lumachad/amd-staging/extended-testing-v2

Conversation

@lumachad

@lumachad lumachad commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

Add six extended test configurations that run in parallel with the
standard tests on every pull request. All jobs reuse the single build
from therock-build-linux (no extra build cost) and fan out together
once the build completes. Extended jobs are PR-only and do not run on
branch pushes.

The extended configurations are:

Job Options
--parallel --parallel
--optimization=-O3 --optimization=-O3 --parallel
--optimization="-O3 -flto" --optimization='-O3 -flto' --parallel
--check-type check-read1 --check-type check-read1 --parallel
--check-type check-readmore --check-type check-readmore --parallel
hip board gdb.base/break.exp --runtestflags=--target_board=hip --tests gdb.base/break.exp (GPU only)

To support per-config scoping, therock-test-packages.yml gains two
new inputs:

  • test_options — appended to each component's test_script (passed
    via a TEST_OPTIONS env var in the jq patch step to avoid injection).
  • projects_to_test — restricts the component matrix; the HIP board
    job uses rocgdb-gpu only since the target board flag is GPU-specific.

🤖 Generated with Claude Code

@lumachad
lumachad requested a review from a team as a code owner August 4, 2026 10:46
@lumachad
lumachad marked this pull request as draft August 4, 2026 10:47
@lumachad lumachad assigned lumachad and unassigned aktemur Aug 4, 2026
@lumachad
lumachad force-pushed the users/lumachad/amd-staging/extended-testing-v2 branch 3 times, most recently from 290e6e7 to c7d4877 Compare August 4, 2026 11:46
@lumachad

lumachad commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

Blocked on failures fixed by #247.

@lumachad lumachad added the ci:skip Skip all pre-commit / CI jobs while the label is up label Aug 5, 2026
@lumachad lumachad assigned lumachad and unassigned lumachad Aug 5, 2026
@lumachad lumachad removed the ci:skip Skip all pre-commit / CI jobs while the label is up label Aug 7, 2026
@lumachad
lumachad force-pushed the users/lumachad/amd-staging/extended-testing-v2 branch from c7d4877 to 249a25a Compare August 7, 2026 06:22
Add six extended test configurations that run in parallel with the
standard tests on every pull request. All jobs reuse the single build
from therock-build-linux (no extra build cost) and fan out together
once the build completes. Extended jobs are PR-only and do not run on
branch pushes.

The extended configurations are:

- --parallel
- --optimization=-O3 --parallel
- --optimization='-O3 -flto' --parallel
- --check-type check-read1 --parallel
- --check-type check-readmore --parallel
- --runtestflags=--target_board=hip --tests gdb.base/break.exp (GPU only)

To support per-config scoping, therock-test-packages.yml gains two new
inputs: test_options, appended to each component's test_script, and
projects_to_test, to restrict the component matrix (the HIP board job
targets rocgdb-gpu only).

Co-Authored-By: Claude <noreply@anthropic.com>
@lumachad
lumachad force-pushed the users/lumachad/amd-staging/extended-testing-v2 branch from 249a25a to c8c9c33 Compare August 7, 2026 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants