Skip to content

Conversation

@rocm-devops
Copy link

Updated GitHub Actions workflows to only run CI checks when targeting
amd-staging, amd-mainline, or amd-npi branches. This prevents unnecessary
CI runs on feature branches and release branches.

Modified workflows:

  • continuous_integration.yml: core-deb, core-rpm, sanitizers jobs
  • code_coverage.yml: code-coverage job
  • rocm_release_compatibility.yml: rocm-release-compatibility job
  • formatting.yml: cmake, source, python, missing-new-line jobs
  • python.yml: linting job
  • restrictions.yml: regex job

All jobs now check if github.base_ref (for PRs) or github.ref (for pushes)
matches one of the allowed primary branches before executing.

PR Details

Associated Jira Ticket Number/Link

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Continuous Integration

Technical details

Added/updated tests?

  • Yes
  • No, Does not apply to this PR.

Updated CHANGELOG?

  • Yes
  • No, Does not apply to this PR.

Added/Updated documentation?

  • Yes
  • No, Does not apply to this PR.

bwelton added 2 commits August 6, 2025 09:47
  Updated GitHub Actions workflows to only run CI checks when targeting
  amd-staging, amd-mainline, or amd-npi branches. This prevents unnecessary
  CI runs on feature branches and release branches.

  Modified workflows:
  - continuous_integration.yml: core-deb, core-rpm, sanitizers jobs
  - code_coverage.yml: code-coverage job
  - rocm_release_compatibility.yml: rocm-release-compatibility job
  - formatting.yml: cmake, source, python, missing-new-line jobs
  - python.yml: linting job
  - restrictions.yml: regex job

  All jobs now check if github.base_ref (for PRs) or github.ref (for pushes)
  matches one of the allowed primary branches before executing.
@rocm-devops
Copy link
Author

Manual review required for cee07b
Assigned Auditors: @anikitin

@rocm-devops
Copy link
Author

Manual review required for 6a4029
Assigned Auditors: @jalan

@jayhawk-commits
Copy link
Contributor

Hitting merge conflict when trying to import to ROCm/rocm-systems:
CONFLICT (modify/delete): projects/rocprofiler-sdk/.github/workflows/formatting.yml deleted in HEAD and modified in 0b2a8e9. Version 0b2a8e9 of projects/rocprofiler-sdk/.github/workflows/formatting.yml left in tree.

ammallya pushed a commit that referenced this pull request Oct 28, 2025
* MI300 Stochastic PC sampling SDK API implementation

* ROCProfV3: Stochastic PC sampling Support (#94)

* ROCProfV3: MI300 Stochastic PC sampling initial draft

* ROCProfV3: Initial Stochastic PC sampling Tests (#95)

ROCProfV3: Initial Stochastic PC sampling tests

* Update rocprofiler_pc_sampling_record_stochastic_v0_t

- update doxygen docs for members
- replace rocprofiler_correlation_id_t with rocprofiler_async_correlation_id_t

* Relax the check in JSON tests

* drain PC sampling buffer during finalize_rocprofv3

* Increase timeout for "Test Install Build" step

- 10 minutes -> 20 minutes
- "Test Installed Packages" has 20 minutes so "Test Install Build" should also

---------

Co-authored-by: Jonathan R. Madsen <jonathanrmadsen@gmail.com>

[ROCm/rocprofiler-sdk commit: 49ce79a]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants