From 05e606a96e87dc1061ee1a95d05257e9915946ab Mon Sep 17 00:00:00 2001 From: Ramakrishna Prabhu Date: Mon, 10 Aug 2026 11:36:29 -0500 Subject: [PATCH 1/2] ci: make pr-test-summary and NVSkills CI checks non-blocking - Remove pr-test-summary from pr-builder needs; it runs independently via if: always() and posts PR comments without gating merge - Remove continue-on-error from pr-test-summary; non-blocking is now expressed by exclusion from pr-builder rather than suppressing failures - Remove continue-on-error from multi-gpu-cpp-tests; now properly blocking through pr-builder as the test suite has matured - Add name "PR test summary (non-blocking)" and "NVSkills CI (non-blocking)" to make advisory status visible in the PR checks UI --- .github/workflows/pr.yaml | 5 +---- .github/workflows/request-nvskills-ci.yml | 1 + 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml index b182380e4c..600f757b78 100644 --- a/.github/workflows/pr.yaml +++ b/.github/workflows/pr.yaml @@ -34,7 +34,6 @@ jobs: - wheel-tests-cuopt-server - wheel-build-cuopt-sh-client - test-self-hosted-server - - pr-test-summary permissions: contents: read uses: rapidsai/shared-workflows/.github/workflows/pr-builder.yaml@main @@ -392,8 +391,6 @@ jobs: if: fromJSON(needs.changed-files.outputs.changed_file_groups).test_cpp with: build_type: pull-request - # Non-blocking while the multi-GPU test suite is maturing. - continue-on-error: true conda-python-build: needs: [build-details, conda-cpp-build, changed-files] # Consumed by conda-python-tests and docs-build. @@ -578,6 +575,7 @@ jobs: build_type: pull-request script: ci/test_self_hosted_service.sh pr-test-summary: + name: "PR test summary (non-blocking)" needs: - conda-cpp-tests - conda-python-tests @@ -585,7 +583,6 @@ jobs: - wheel-tests-cuopt-server - test-self-hosted-server if: always() - continue-on-error: true runs-on: ubuntu-latest permissions: actions: read diff --git a/.github/workflows/request-nvskills-ci.yml b/.github/workflows/request-nvskills-ci.yml index da28f21ded..3bc865b526 100644 --- a/.github/workflows/request-nvskills-ci.yml +++ b/.github/workflows/request-nvskills-ci.yml @@ -21,6 +21,7 @@ permissions: jobs: require-nvskills-ci: + name: "NVSkills CI (non-blocking)" if: github.event_name == 'pull_request' permissions: contents: read From 6b60a0303089d91104ad6117ea7dae7cdb7906ac Mon Sep 17 00:00:00 2001 From: Ramakrishna Prabhu Date: Mon, 10 Aug 2026 13:14:46 -0500 Subject: [PATCH 2/2] ci: exclude pr-test-summary from pr-builder dependency check --- .github/workflows/pr.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml index 600f757b78..d54afeebd4 100644 --- a/.github/workflows/pr.yaml +++ b/.github/workflows/pr.yaml @@ -340,6 +340,7 @@ jobs: uses: rapidsai/shared-workflows/.github/workflows/checks.yaml@main with: enable_check_generated_files: false + ignored_pr_jobs: "pr-test-summary" conda-cpp-build: needs: [build-details, checks, compute-matrix-filters, changed-files] # Consumed by conda-cpp-tests, conda-python-build, and (transitively) docs-build.