Skip to content

Commit

Permalink
PR CI: make cugraph builds depend on pylibcugraph builds (#4801)
Browse files Browse the repository at this point in the history
Proposes the following change to the wheel jobs for PR CI:

```mermaid
---
title: Current
---
flowchart LR
    A[build-pylibcugraph] --> B[test-pylibcugraph]
    B --> C[build-cugraph]
    C --> D[test-cugraph]
```

```mermaid
---
title: Proposed
---
flowchart LR
    A[build-pylibcugraph] --> B[test-pylibcugraph]
    A --> C[build-cugraph]
    C --> D[test-cugraph]
```

## Notes for Reviewers

I think reducing the end-to-end time for changes here is even more important now that we have new downstream repos (https://github.com/rapidsai/nx-cugraph and https://github.com/rapidsai/cugraph-gnn) that depend on changes made here.

### Benefits of this change

* shorter end-to-end time for CI runs
  - *by parallelizing more work*
  - *by doing `cugraph-cu{11,12}` builds more frequently, which should mean more frequent population of the `sccache` cache*
* faster feedback about `cugraph-cu{11,12}` build issues
* consistency with the rest of RAPIDS (every other RAPIDS project has builds depend on builds, not tests, as far as I know)

### Costs of this change

* more at-one-moment load on GPU CI runners as a result of `cugraph` PRs (wheel tests for `cugraph` could now occupy 4 GPU runners at once instead of just 2)
  - *`pylibcugraph` test jobs tend to take around 6-10 minutes once scheduled onto a runner, so this shouldn't be too bad*

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

Approvers:
  - Chuck Hastings (https://github.com/ChuckHastings)
  - Bradley Dice (https://github.com/bdice)
  - Alex Barghi (https://github.com/alexbarghi-nv)

URL: #4801
  • Loading branch information
jameslamb authored Dec 9, 2024
1 parent 5c8f850 commit de05abd
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ jobs:
build_type: pull-request
script: ci/test_wheel_pylibcugraph.sh
wheel-build-cugraph:
needs: wheel-tests-pylibcugraph
needs: wheel-build-pylibcugraph
secrets: inherit
uses: rapidsai/shared-workflows/.github/workflows/wheels-build.yaml@branch-25.02
with:
Expand Down
1 change: 1 addition & 0 deletions python/cugraph/pytest.ini
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,7 @@ filterwarnings =
error::FutureWarning
error::DeprecationWarning
# TODO
ignore:.*cuda..* module is deprecated.*:DeprecationWarning
ignore:Multi is deprecated and the removal of multi edges will no longer be supported:FutureWarning
ignore:The legacy column names:FutureWarning
ignore:The include_hop_column flag is deprecated and will be removed:FutureWarning
Expand Down

0 comments on commit de05abd

Please sign in to comment.