Skip to content

[TheRock] CDash Reporting for rocprofiler-sdk tests#4409

Open
EvanPhoukong wants to merge 64 commits into
mainfrom
users/ephoukong/run-therock-ci
Open

[TheRock] CDash Reporting for rocprofiler-sdk tests#4409
EvanPhoukong wants to merge 64 commits into
mainfrom
users/ephoukong/run-therock-ci

Conversation

@EvanPhoukong

@EvanPhoukong EvanPhoukong commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Motivation

Report rocprofiler-sdk configure, build, and test results from TheRock CI to CDash while preserving the existing local test flow.

JIRA Ticket

AIPROFSDK-34

Technical Details

  • Adds an optional --enable-cdash execution path and enables it for rocprofiler-sdk CI.
  • Generates a CTest dashboard using the same configure, build, and test commands as the direct runner.
  • Includes PR, platform, GPU, build variant, artifact run, and test run information in CDash submissions.
  • Preserves ASan exclusions and supports optional strict submission.
  • Keeps direct CMake and CTest execution as the default for local use.

Test Plan

  • Run the dashboard unit tests.
  • Confirm CI submits configure, build, and test results with the expected metadata.
  • Confirm local execution still uses the direct runner.

Test Result

  • rocprofiler_sdk_dashboard_test.py: 8 passed.
  • Confirmed PR submission is correct in the Dashboard

Submission Checklist

@EvanPhoukong EvanPhoukong changed the title Users/ephoukong/run therock ci [TheRock] CDash Reporting for TheRock Apr 8, 2026
@EvanPhoukong EvanPhoukong added CICD Related to CI/CD Infra test:rocprim For pull requests, runs full tests for only rocprim and other labeled projects. WIP TheRock and removed test:rocprim For pull requests, runs full tests for only rocprim and other labeled projects. labels Apr 13, 2026
@EvanPhoukong
EvanPhoukong requested a review from a team as a code owner April 17, 2026 17:40
@EvanPhoukong
EvanPhoukong removed the request for review from a team April 17, 2026 17:50
@EvanPhoukong EvanPhoukong added test case pending Relevant test case is still needed to ensure issue doesn't reappear. do-not-review and removed WIP labels Apr 17, 2026
@EvanPhoukong EvanPhoukong changed the title [TheRock] CDash Reporting for TheRock [Do Not Review] [TheRock] CDash Reporting for TheRock Apr 30, 2026
@marbre
marbre marked this pull request as draft May 4, 2026 18:13
@marbre

marbre commented May 4, 2026

Copy link
Copy Markdown
Member

As you touched modules it will trigger a review request anyway. Please mark this as minimum a draft. However, you probably don't need a PR at all and you can rather trigger a specific action you want to run instead.

@EvanPhoukong
EvanPhoukong force-pushed the users/ephoukong/run-therock-ci branch 3 times, most recently from e3a5337 to 13ce67f Compare May 6, 2026 14:38
@EvanPhoukong EvanPhoukong changed the title [Do Not Review] [TheRock] CDash Reporting for TheRock [TheRock] CDash Reporting for TheRock May 6, 2026
@EvanPhoukong EvanPhoukong added ready-for-first-review and removed test case pending Relevant test case is still needed to ensure issue doesn't reappear. do-not-review labels May 6, 2026
@EvanPhoukong
EvanPhoukong requested a review from venkat1361 May 6, 2026 18:35
@EvanPhoukong
EvanPhoukong force-pushed the users/ephoukong/run-therock-ci branch 2 times, most recently from 3bdd10d to 2099637 Compare May 10, 2026 03:09
@EvanPhoukong
EvanPhoukong marked this pull request as ready for review May 11, 2026 14:04
Comment thread build_tools/github_actions/fetch_test_configurations.py Outdated
Comment thread build_tools/github_actions/test_executable_scripts/test_rocprofiler_sdk.py Outdated
Comment thread build_tools/github_actions/test_executable_scripts/test_rocprofiler_sdk.py Outdated
Comment thread build_tools/github_actions/test_executable_scripts/test_rocprofiler_sdk.py Outdated
@EvanPhoukong
EvanPhoukong requested a review from venkat1361 May 11, 2026 20:05
@EvanPhoukong
EvanPhoukong force-pushed the users/ephoukong/run-therock-ci branch from fd1606c to c8db38d Compare July 21, 2026 22:40
@therock-pr-bot

therock-pr-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

❌ PR Check — Action Required

Check Status Details
🌿 Branch Name ✅ Pass
📝 PR Title/Description ❌ Fail Error: PR description must reference a JIRA ID, ISSUE ID, or a GitHub closing keyword.
Expected: include a JIRA ID / ISSUE ID line (separator : or -, or omitted; value may be a JIRA key, a number with/without #, or a link), OR a closing keyword + issue reference. Accepted examples:
JIRA ID : TESTAUTO-6039
JIRA ID - #330
JIRA ID #330
JIRA ID (on separate line)
ROCM-25757
ISSUE ID : TESTUTO-3334
ISSUE ID #3334
ISSUE ID - TESTAUTO-3433
ISSUE ID (on separate line)
AIRUNTIME-2352
ISSUE ID : https://github.com/<org_name>/<repo_name>/issues/1234
Closes #10
Fixes octo-org/octo-repo#100
Resolves: #123
#123
https://github.com/<org_name>/<repo_name>/issues/123
Current: no valid JIRA/ISSUE/closing-keyword reference found
Forbidden Files ✅ Pass
🧪 Unit Test ✅ Pass
🔎 pre-commit ⏳ Pending ⏳ Still running…
🚫 Draft PR 🔜 To Be Enabled
🚩 Feature Flag 🔜 To Be Enabled
📊 Code Coverage 🔜 To Be Enabled

⚠️ 1 policy check(s) failed. Please address the issues above before this PR can be Reviewed.

🚫 Please fix the failed policies

  • ❌ PR Title/Description

The Not ready to Review label was added to this PR. Once all policies pass, the label is removed automatically.

📖 Need help? See the Policy FAQ for details on every check and how to fix failures.

@therock-pr-bot therock-pr-bot Bot added the Not ready to Review PR has unresolved policy failures — reviews blocked label Jul 21, 2026
@therock-pr-bot

therock-pr-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

🚫 Please fix the failed policies before requesting reviews.

The following policy checks failed:

  • ❌ PR Title/Description

The Not ready to Review label has been added to this PR.
Once all policies pass, the label will be removed automatically.

@therock-pr-bot therock-pr-bot Bot removed the Not ready to Review PR has unresolved policy failures — reviews blocked label Jul 23, 2026
@EvanPhoukong
EvanPhoukong requested a review from geomin12 July 23, 2026 19:38
@therock-pr-bot therock-pr-bot Bot added the Not ready to Review PR has unresolved policy failures — reviews blocked label Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Related to CI/CD Infra Not ready to Review PR has unresolved policy failures — reviews blocked ready-for-first-review

Projects

Status: TODO

Development

Successfully merging this pull request may close these issues.

5 participants