Skip to content

Always Enable Beta PC Sampling in Internal CI - #1131

Closed
vlaindic wants to merge 1 commit into
developfrom
users/vlaindic/always-enable-beta-pc-sampling
Closed

vlaindic wants to merge 1 commit into
developfrom
users/vlaindic/always-enable-beta-pc-sampling

Conversation

@vlaindic

@vlaindic vlaindic commented Sep 25, 2025 •

Copy link
Copy Markdown
Member

Internal CI will try running all PC sampling tests. Guards for skipping tests if PC sampling is unsupported are embedded in testing suite.

This PR supersedes the #1076 .

Motivation

To increase PC sampling robustness, we always enable beta PC sampling in rocprofiler's CI tests.

Technical Details

If PC sampling is unsupported, then:

  • On some architectures, we disabled tests in compile time.
  • On others, tests will run and emit message for skipping a test.

Test Plan

Test Result

Submission Checklist

@vlaindic
vlaindic requested a review from a team as a code owner September 25, 2025 10:59
@github-actions github-actions Bot added the github actions Pull requests that update GitHub Actions code label Sep 25, 2025
@vlaindic
vlaindic force-pushed the users/vlaindic/always-enable-beta-pc-sampling branch 3 times, most recently from 5723a6d to 8c0f5f7 Compare October 7, 2025 08:27
@vlaindic
vlaindic marked this pull request as draft November 4, 2025 16:52
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been inactive for 25 days and will be marked as stale.

If you would like to keep this PR open, please:

  • Add new commits
  • Add a comment explaining why it should remain open

This PR will be automatically closed in 5 days if no further activity occurs.

@github-actions github-actions Bot added the Stale PR has no activity for 25+ days label May 14, 2026
Internal CI will try running all PC sampling tests.
Guards for skipping tests if PC sampling is unsupported
are embedded in testing suite.
@vlaindic
vlaindic force-pushed the users/vlaindic/always-enable-beta-pc-sampling branch from 8c0f5f7 to 9a26ea8 Compare May 15, 2026 11:17
@github-actions github-actions Bot removed the Stale PR has no activity for 25+ days label May 16, 2026
@vlaindic

Copy link
Copy Markdown
Member Author

Still not safe to apply yet due to AILIROCR-510 and ROCM-22213

@vlaindic vlaindic closed this May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github actions Pull requests that update GitHub Actions code organization: ROCm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant