Skip to content

Commit f0015ac

Browse files
committed
ci: exercise release build output in PRs
1 parent bdeac96 commit f0015ac

2 files changed

Lines changed: 35 additions & 35 deletions

File tree

.github/workflows/build.yaml

Lines changed: 8 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ jobs:
7070
packages: read
7171
pull-requests: read
7272
secrets: inherit # zizmor: ignore[secrets-inherit]
73-
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@main
73+
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@codex/release-build-output-manifests
7474
strategy:
7575
fail-fast: false
7676
matrix: ${{ fromJSON(needs.rocky8-clib-standalone-build-matrix.outputs.matrix) }}
@@ -87,22 +87,12 @@ jobs:
8787
file_to_upload: |
8888
libcuvs_c.tar.gz
8989
libcuvs_c.release-package.json
90-
sha: ${{ inputs.sha }}
91-
rocky8-clib-standalone-release-output:
92-
needs: [rocky8-clib-standalone-build-matrix, rocky8-clib-standalone-build]
93-
permissions:
94-
actions: read
95-
contents: read
96-
uses: rapidsai/shared-workflows/.github/workflows/release-build-output.yaml@codex/release-build-output-manifests
97-
strategy:
98-
fail-fast: false
99-
matrix: ${{ fromJSON(needs.rocky8-clib-standalone-build-matrix.outputs.matrix) }}
100-
with:
101-
artifact-name: "libcuvs_c_${{ matrix.CUDA_VER }}_${{ matrix.ARCH }}.tar.gz"
90+
release-build-output: true
91+
release-output-directory: .
10292
release-unit: archive:libcuvs-c
10393
release-package-file: libcuvs_c.release-package.json
10494
release-artifacts: '[{"path":"libcuvs_c.tar.gz"}]'
105-
source-sha: ${{ inputs.sha || github.sha }}
95+
sha: ${{ inputs.sha }}
10696
rust-build-matrix:
10797
needs: cpp-build
10898
permissions:
@@ -187,7 +177,7 @@ jobs:
187177
packages: read
188178
pull-requests: read
189179
secrets: inherit # zizmor: ignore[secrets-inherit]
190-
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@main
180+
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@codex/release-build-output-manifests
191181
# Artifacts are not published from these jobs, so it's safe to run for multiple CUDA versions.
192182
# If these jobs start producing artifacts, the names will have to differentiate between CUDA versions.
193183
strategy:
@@ -202,22 +192,12 @@ jobs:
202192
script: "ci/build_java.sh"
203193
artifact-name: "cuvs-java-cuda${{ matrix.CUDA_VER }}"
204194
file_to_upload: "java/cuvs-java/target/"
205-
sha: ${{ inputs.sha }}
206-
java-release-output:
207-
needs: [java-build-matrix, java-build]
208-
permissions:
209-
actions: read
210-
contents: read
211-
uses: rapidsai/shared-workflows/.github/workflows/release-build-output.yaml@codex/release-build-output-manifests
212-
strategy:
213-
fail-fast: false
214-
matrix: ${{ fromJSON(needs.java-build-matrix.outputs.matrix) }}
215-
with:
216-
artifact-name: "cuvs-java-cuda${{ matrix.CUDA_VER }}"
195+
release-build-output: true
196+
release-output-directory: java/cuvs-java/target
217197
release-unit: maven:cuvs-java
218198
release-package-file: cuvs-java.release-package.json
219199
release-artifacts: '[{"path":"cuvs-java-*-x86_64-cuda*.jar"}]'
220-
source-sha: ${{ inputs.sha || github.sha }}
200+
sha: ${{ inputs.sha }}
221201
python-build:
222202
needs: [cpp-build]
223203
permissions:

.github/workflows/pr.yaml

Lines changed: 27 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -399,11 +399,13 @@ jobs:
399399
packages: read
400400
pull-requests: read
401401
secrets: inherit # zizmor: ignore[secrets-inherit]
402-
uses: rapidsai/shared-workflows/.github/workflows/conda-cpp-build.yaml@main
402+
uses: rapidsai/shared-workflows/.github/workflows/conda-cpp-build.yaml@codex/release-build-output-manifests
403403
with:
404404
build_type: pull-request
405405
node_type: cpu16
406406
script: ci/build_cpp.sh
407+
release-build-output: true
408+
release-unit: conda:cuvs-cpp
407409
conda-cpp-tests:
408410
needs: [conda-cpp-build, changed-files]
409411
permissions:
@@ -440,10 +442,12 @@ jobs:
440442
packages: read
441443
pull-requests: read
442444
secrets: inherit # zizmor: ignore[secrets-inherit]
443-
uses: rapidsai/shared-workflows/.github/workflows/conda-python-build.yaml@main
445+
uses: rapidsai/shared-workflows/.github/workflows/conda-python-build.yaml@codex/release-build-output-manifests
444446
with:
445447
build_type: pull-request
446448
script: ci/build_python.sh
449+
release-build-output: true
450+
release-unit: conda:cuvs-python
447451
# Build a conda package for each CUDA x ARCH x minimum supported Python version
448452
matrix_filter: group_by({CUDA_VER, ARCH}) | map(min_by(.PY_VER | split(".") | map(tonumber)))
449453
conda-python-tests:
@@ -477,7 +481,7 @@ jobs:
477481
packages: read
478482
pull-requests: read
479483
secrets: inherit # zizmor: ignore[secrets-inherit]
480-
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@main
484+
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@codex/release-build-output-manifests
481485
strategy:
482486
fail-fast: false
483487
matrix: ${{ fromJSON(needs.rocky8-clib-standalone-build-matrix.outputs.matrix) }}
@@ -490,7 +494,14 @@ jobs:
490494
requires_license_builder: true
491495
script: "ci/build_standalone_c.sh --build-tests"
492496
artifact-name: "libcuvs_c_${{ matrix.CUDA_VER }}_${{ matrix.ARCH }}.tar.gz"
493-
file_to_upload: "libcuvs_c.tar.gz"
497+
file_to_upload: |
498+
libcuvs_c.tar.gz
499+
libcuvs_c.release-package.json
500+
release-build-output: true
501+
release-output-directory: .
502+
release-unit: archive:libcuvs-c
503+
release-package-file: libcuvs_c.release-package.json
504+
release-artifacts: '[{"path":"libcuvs_c.tar.gz"}]'
494505
sha: ${{ inputs.sha }}
495506
rocky8-clib-tests-matrix:
496507
needs: [rocky8-clib-standalone-build, changed-files]
@@ -541,7 +552,7 @@ jobs:
541552
packages: read
542553
pull-requests: read
543554
secrets: inherit # zizmor: ignore[secrets-inherit]
544-
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@main
555+
uses: rapidsai/shared-workflows/.github/workflows/custom-job.yaml@codex/release-build-output-manifests
545556
# Artifacts are not published from these jobs, so it's safe to run for multiple CUDA versions.
546557
# If these jobs start producing artifacts, the names will have to differentiate between CUDA versions.
547558
strategy:
@@ -555,6 +566,11 @@ jobs:
555566
script: "ci/test_java.sh"
556567
artifact-name: "cuvs-java-cuda${{ matrix.CUDA_VER }}"
557568
file_to_upload: "java/cuvs-java/target/"
569+
release-build-output: true
570+
release-output-directory: java/cuvs-java/target
571+
release-unit: maven:cuvs-java
572+
release-package-file: cuvs-java.release-package.json
573+
release-artifacts: '[{"path":"cuvs-java-*-x86_64-cuda*.jar"}]'
558574
rust-build-matrix:
559575
needs: [conda-cpp-build, changed-files]
560576
permissions:
@@ -641,7 +657,7 @@ jobs:
641657
packages: read
642658
pull-requests: read
643659
secrets: inherit # zizmor: ignore[secrets-inherit]
644-
uses: rapidsai/shared-workflows/.github/workflows/wheels-build.yaml@main
660+
uses: rapidsai/shared-workflows/.github/workflows/wheels-build.yaml@codex/release-build-output-manifests
645661
with:
646662
build_type: pull-request
647663
node_type: cpu16
@@ -650,6 +666,8 @@ jobs:
650666
matrix_filter: group_by([.ARCH, (.CUDA_VER|split(".")|map(tonumber)|.[0])]) | map(max_by(.PY_VER|split(".")|map(tonumber)))
651667
package-name: libcuvs
652668
package-type: cpp
669+
release-build-output: true
670+
release-unit: wheel:libcuvs
653671
wheel-build-cuvs:
654672
needs: wheel-build-libcuvs
655673
permissions:
@@ -659,13 +677,15 @@ jobs:
659677
packages: read
660678
pull-requests: read
661679
secrets: inherit # zizmor: ignore[secrets-inherit]
662-
uses: rapidsai/shared-workflows/.github/workflows/wheels-build.yaml@main
680+
uses: rapidsai/shared-workflows/.github/workflows/wheels-build.yaml@codex/release-build-output-manifests
663681
with:
664682
build_type: pull-request
665683
node_type: cpu8
666684
script: ci/build_wheel_cuvs.sh
667685
package-name: cuvs
668686
package-type: python
687+
release-build-output: true
688+
release-unit: wheel:cuvs
669689
# Build a wheel for each CUDA x ARCH x minimum supported Python version
670690
matrix_filter: group_by({CUDA_VER, ARCH}) | map(min_by(.PY_VER | split(".") | map(tonumber)))
671691
wheel-tests-cuvs:

0 commit comments

Comments
 (0)