Skip to content

DAOS-18626 ci: Add TSan report parser (SARIF + job summary) - #19102

Draft
knard38 wants to merge 1 commit into
masterfrom
ckochhof/dev/master/daos-18626/patch-006
Draft

knard38 wants to merge 1 commit into
masterfrom
ckochhof/dev/master/daos-18626/patch-006

Conversation

@knard38

@knard38 knard38 commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

TODO

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).

Add utils/ci/parse_tsan_reports.py, which parses ThreadSanitizer
tsan.<pid> log files (produced with
TSAN_OPTIONS="log_path=<dir>/tsan:exitcode=42:second_deadlock_stack=1")
and emits:

- A SARIF 2.1.0 file for upload to the GitHub Security / Code scanning
  tab, using the shared helpers from sanitizer_report_base.py.
- A Markdown job-summary table (test, error type, primary location,
  description) with an Argobots-false-positive note, plus a collapsible
  per-report details section with the raw captured log.

TSan reports are parsed into a nested thread/frames structure (unlike
ASan/UBSan's flat frame list) since each report can describe multiple
racing thread stacks.

Not yet invoked by any workflow or script. Part of the DAOS-18626 ASan
CI split (see ckochhof/dev/master/daos-18626/patch-001 for the
original, unsplit commit).

Signed-off-by: Cedric Koch-Hofer <cedric.koch-hofer@hpe.com>
@github-actions

Copy link
Copy Markdown

Ticket title is 'Add ASan/UBSan/TSan sanitizer checks to the GitHub Actions unit-test CI'
Status is 'In Progress'
https://daosio.atlassian.net/browse/DAOS-18626

@knard38 knard38 self-assigned this Sep 17, 2026
@daosbuild3

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants