From de05abd8fae510e5b96d634523c2f11efecbeb4a Mon Sep 17 00:00:00 2001
From: James Lamb <jlamb@nvidia.com>
Date: Mon, 9 Dec 2024 11:04:50 -0600
Subject: [PATCH] PR CI: make cugraph builds depend on pylibcugraph builds
 (#4801)

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: https://github.com/rapidsai/cugraph/pull/4801
---
 .github/workflows/pr.yaml | 2 +-
 python/cugraph/pytest.ini | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml
index 633d964a3b..c9e41475a1 100644
--- a/.github/workflows/pr.yaml
+++ b/.github/workflows/pr.yaml
@@ -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:
diff --git a/python/cugraph/pytest.ini b/python/cugraph/pytest.ini
index bf6e6bdd80..7bdef2d977 100644
--- a/python/cugraph/pytest.ini
+++ b/python/cugraph/pytest.ini
@@ -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