Skip to content

[Ex CI] create centralized Azure CI dispatcher workflow#355

Merged
danielsu-amd merged 2 commits into
ROCm:developfrom
danielsu-amd:users/danielsu/az-ci-dispatch
Jun 24, 2025
Merged

[Ex CI] create centralized Azure CI dispatcher workflow#355
danielsu-amd merged 2 commits into
ROCm:developfrom
danielsu-amd:users/danielsu/az-ci-dispatch

Conversation

@danielsu-amd
Copy link
Copy Markdown
Contributor

Creates an azure-ci-dispatcher Github Action which is intended to be the one-stop shop for Azure CI runs on PRs. This action is intended to completely replace the existing Azure CI triggers once verified to be fully functional. It allows for:

  • A single check encompassing all Azure CI runs, allows only one 'required' check to be set
  • Skipped Azure runs will not be shown in a PR's check list
  • Much more flexibility on triggering Azure CI, since Github Actions have better integration with the rest of the website
    • ex. Can trigger Azure when drafts are set to 'ready for review'
    • ex. Can trigger Azure with a label (not implemented in here)

The action is 'smarter' and will skip CI runs that are already covered by an upstream dependency's run.

  • ex. a PR touching rocPRIM and rocThrust will only trigger a rocPRIM run, which already will build rocThrust as part of its downstream jobs.

Once runs are started, the action will periodically query the Azure API for the jobs' statuses. Once all jobs are finished, the action will exit successfully if all jobs are successful, or will exit unsuccessfully if any jobs are unsuccessful.

Some test runs:

This currently does not have any functionality for include/exclude file patterns (these fields), any suggestions on how we could do that are welcome.

@danielsu-amd danielsu-amd requested review from a team, eidenyoshida and rahulc-gh June 24, 2025 15:45
@danielsu-amd danielsu-amd self-assigned this Jun 24, 2025
@danielsu-amd danielsu-amd merged commit 607229d into ROCm:develop Jun 24, 2025
11 checks passed
ammallya pushed a commit that referenced this pull request Sep 18, 2025
rahulc-gh pushed a commit that referenced this pull request Sep 23, 2025
ammallya pushed a commit that referenced this pull request Nov 5, 2025
Bumps [cryptography](https://github.com/pyca/cryptography) from 42.0.7 to 44.0.1.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@42.0.7...44.0.1)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-version: 44.0.1
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
ammallya pushed a commit that referenced this pull request Nov 5, 2025
Bumps [cryptography](https://github.com/pyca/cryptography) from 42.0.7 to 44.0.1.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@42.0.7...44.0.1)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-version: 44.0.1
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

[ROCm/hipTensor commit: 4c83d6b]
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.

1 participant