Skip to content

DAOS-19106 cq: auto test tags for pytorch and cart#18462

Open
daltonbohning wants to merge 1 commit into
masterfrom
dbohning/daos-19106
Open

DAOS-19106 cq: auto test tags for pytorch and cart#18462
daltonbohning wants to merge 1 commit into
masterfrom
dbohning/daos-19106

Conversation

@daltonbohning

Copy link
Copy Markdown
Contributor

Map src/cart -> pr cart
Map ftest/cart/*.[ch] -> cart
Map pydaos/torch -> pytorch

Doc-only: true

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

Map src/cart -> pr cart
Map ftest/cart/*.[ch] -> cart
Map pydaos/torch -> pytorch

Doc-only: true

Signed-off-by: Dalton Bohning <dalton.bohning@hpe.com>
@daltonbohning daltonbohning self-assigned this Jun 8, 2026
@daltonbohning daltonbohning marked this pull request as ready for review June 8, 2026 16:27
Comment on lines +21 to +25
# Run only cart ftest for cart ftest C utilities
src/tests/ftest/cart/*.[ch]:
test-tag: cart
<<: *ftest-only

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@frostedcmos This would mean that if you only modified an ftest cart utility, CI will automatically skip unit tests and non-cart PR tests.

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

Ticket title is 'ftest: auto test tag for pytorch and cart'
Status is 'In Review'
https://daosio.atlassian.net/browse/DAOS-19106

@enakta enakta 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, very nice !

@daltonbohning daltonbohning added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed.

Development

Successfully merging this pull request may close these issues.

3 participants