Skip to content

Conversation

@jameslamb
Copy link
Member

@jameslamb jameslamb commented Aug 4, 2025

Fixes #769

Notes for Reviewers

How this works

In this repo, we have a pattern roughly like this for CI:

# pr.yaml
name: ci
...
jobs:
  docker:
    uses: ./.github/workflows/build-test-publish-images.yml
# build-test-publish-images.yaml
name: build and publish imgs workflow
...
jobs:
  build-rapids:
    strategy:
      matrix: ${{ fromJSON(needs.compute-matrix.outputs.MATRIX) }}
    uses: ./.github/workflows/build-rapids-image.yml
# build-rapids-image.yml
name: Build and push image variant
...
jobs:
  build:
    strategy:
      matrix:
        ARCH: ${{ fromJSON(inputs.ARCHES) }}
        CUDA_VER: ["${{ inputs.CUDA_VER }}"]
        LINUX_VER: ["${{ inputs.LINUX_VER }}"]
        PYTHON_VER: ["${{ inputs.PYTHON_VER }}"]
        RAPIDS_VER: ["${{ inputs.RAPIDS_VER }}"]
...

Each stage in that chain of workflow calls adds something to the full job name. Like this:

Screenshot 2025-08-04 at 4 59 16 PM

ref: https://github.com/rapidsai/docker/actions/runs/16297798451/job/46024115023

Screenshot 2025-08-04 at 5 05 21 PM

ref: #774

@jameslamb jameslamb changed the title WIP: clarify CI job names clarify CI job names Aug 5, 2025
@jameslamb jameslamb marked this pull request as ready for review August 5, 2025 21:28
@jameslamb jameslamb requested a review from a team as a code owner August 5, 2025 21:28
@jameslamb jameslamb requested a review from gforsyth August 5, 2025 21:28
Copy link
Contributor

@gforsyth gforsyth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! This makes the job summary much clearer. And thanks for the excellent primer in the PR description.

:shipit:

@jameslamb
Copy link
Member Author

Thanks for reviewing!

@jameslamb
Copy link
Member Author

/merge

@rapids-bot rapids-bot bot merged commit 0c68cdc into rapidsai:branch-25.10 Aug 6, 2025
60 checks passed
@jameslamb jameslamb deleted the ci-job-names branch August 6, 2025 14:23
rapids-bot bot pushed a commit that referenced this pull request Aug 6, 2025
Fixes some small things noticed while working on #769 

* ~ensures `build-cuvs-image.yml` and `builds-rapids-image.yml` runs have display names~
  - *(this only really matters on the https://github.com/rapidsai/docker/actions page)*
  - *this ended up on #777*
* removes unused inputs `LINUX_DISTRO` and `LINUX_DISTRO_VER` to `build-cuvs`
* updates `pre-commit` hooks to their latest versions (replacing #778)

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Bradley Dice (https://github.com/bdice)

URL: #779
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.

[BUG] CI job names are hard to understand

2 participants