Skip to content

Add test filter standardisation to rocrtst#4812

Draft
dileepr1 wants to merge 8 commits intomainfrom
users/dravindr/tr_rocr
Draft

Add test filter standardisation to rocrtst#4812
dileepr1 wants to merge 8 commits intomainfrom
users/dravindr/tr_rocr

Conversation

@dileepr1
Copy link
Copy Markdown
Contributor

Motivation

Technical Details

Test Plan

Test Result

Submission Checklist

dileepr1 added a commit to ROCm/rocm-systems that referenced this pull request May 6, 2026
## Motivation
The goal is to categorize tests at the component level, enabling TheRock
CI to invoke a common test_runner with specified categories to run the
appropriate tests.

More details available at
[test_filtering.md](https://github.com/ROCm/TheRock/blob/main/docs/development/test_filtering.md).

## Technical Details

- These following are the changes made in rocm-systems:
- Add test_categories.yaml in rocrtst
- Add the parser - parse_test_categories.py, and the associated the
function call in associated TestCategories.cmake at
rocm-systems/shared/ctest
- The framework to make the parser call and create ctest entries and
copy CTestTestfile.cmake over to 'bin/rocrtst'

This works together with the companion PR
[#4812](ROCm/TheRock#4812) at TheRock side.

## Test Plan

Confirm rocrtst is run with correct filters for the corresponding
categories.

## Test Result

Sample run:
https://github.com/ROCm/TheRock/actions/runs/24973684590/job/73132282952?pr=4812

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
saleelk pushed a commit to ROCm/rocm-systems that referenced this pull request May 7, 2026
## Motivation
The goal is to categorize tests at the component level, enabling TheRock
CI to invoke a common test_runner with specified categories to run the
appropriate tests.

More details available at
[test_filtering.md](https://github.com/ROCm/TheRock/blob/main/docs/development/test_filtering.md).

## Technical Details

- These following are the changes made in rocm-systems:
- Add test_categories.yaml in rocrtst
- Add the parser - parse_test_categories.py, and the associated the
function call in associated TestCategories.cmake at
rocm-systems/shared/ctest
- The framework to make the parser call and create ctest entries and
copy CTestTestfile.cmake over to 'bin/rocrtst'

This works together with the companion PR
[#4812](ROCm/TheRock#4812) at TheRock side.

## Test Plan

Confirm rocrtst is run with correct filters for the corresponding
categories.

## Test Result

Sample run:
https://github.com/ROCm/TheRock/actions/runs/24973684590/job/73132282952?pr=4812

## Submission Checklist

- [ ] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
dileepr1 and others added 2 commits May 7, 2026 10:16
Restore rocm-systems gitlink to main's commit so this branch follows
main's behavior instead of pointing to a different location.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: TODO

Development

Successfully merging this pull request may close these issues.

1 participant