Skip to content

Commit 56bd1c3

Browse files
Copilotleofang
andcommitted
Revert ci.yml changes and remove proxy cache from build-docs.yml
Co-authored-by: leofang <5534781+leofang@users.noreply.github.com>
1 parent 32e30f4 commit 56bd1c3

2 files changed

Lines changed: 42 additions & 46 deletions

File tree

.github/workflows/build-docs.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,7 @@ jobs:
5252
fetch-depth: 0
5353
ref: ${{ inputs.git-tag }}
5454

55-
# Temporarily disabled to debug nvidia-sphinx-theme installation issues
56-
# - name: Setup proxy cache
57-
# uses: nv-gha-runners/setup-proxy-cache@main
58-
# continue-on-error: true
55+
# TODO: This workflow runs on GH-hosted runner and cannot use the proxy cache
5956

6057
- name: Set up miniforge
6158
uses: conda-incubator/setup-miniconda@835234971496cad1653abb28a638a281cf32541f # v3.2.0

.github/workflows/ci.yml

Lines changed: 41 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -50,47 +50,46 @@ jobs:
5050
host-platform: ${{ matrix.host-platform }}
5151
cuda-version: ${{ needs.ci-vars.outputs.CUDA_BUILD_VER }}
5252

53-
# Temporarily disabled test jobs for documentation theme migration
54-
# test-linux:
55-
# strategy:
56-
# fail-fast: false
57-
# matrix:
58-
# host-platform:
59-
# - linux-64
60-
# - linux-aarch64
61-
# name: Test ${{ matrix.host-platform }}
62-
# if: ${{ github.repository_owner == 'nvidia' }}
63-
# permissions:
64-
# contents: read # This is required for actions/checkout
65-
# needs:
66-
# - ci-vars
67-
# - build
68-
# secrets: inherit
69-
# uses: ./.github/workflows/test-wheel-linux.yml
70-
# with:
71-
# build-type: pull-request
72-
# host-platform: ${{ matrix.host-platform }}
73-
# build-ctk-ver: ${{ needs.ci-vars.outputs.CUDA_BUILD_VER }}
53+
test-linux:
54+
strategy:
55+
fail-fast: false
56+
matrix:
57+
host-platform:
58+
- linux-64
59+
- linux-aarch64
60+
name: Test ${{ matrix.host-platform }}
61+
if: ${{ github.repository_owner == 'nvidia' }}
62+
permissions:
63+
contents: read # This is required for actions/checkout
64+
needs:
65+
- ci-vars
66+
- build
67+
secrets: inherit
68+
uses: ./.github/workflows/test-wheel-linux.yml
69+
with:
70+
build-type: pull-request
71+
host-platform: ${{ matrix.host-platform }}
72+
build-ctk-ver: ${{ needs.ci-vars.outputs.CUDA_BUILD_VER }}
7473

75-
# test-windows:
76-
# strategy:
77-
# fail-fast: false
78-
# matrix:
79-
# host-platform:
80-
# - win-64
81-
# name: Test ${{ matrix.host-platform }}
82-
# if: ${{ github.repository_owner == 'nvidia' }}
83-
# permissions:
84-
# contents: read # This is required for actions/checkout
85-
# needs:
86-
# - ci-vars
87-
# - build
88-
# secrets: inherit
89-
# uses: ./.github/workflows/test-wheel-windows.yml
90-
# with:
91-
# build-type: pull-request
92-
# host-platform: ${{ matrix.host-platform }}
93-
# build-ctk-ver: ${{ needs.ci-vars.outputs.CUDA_BUILD_VER }}
74+
test-windows:
75+
strategy:
76+
fail-fast: false
77+
matrix:
78+
host-platform:
79+
- win-64
80+
name: Test ${{ matrix.host-platform }}
81+
if: ${{ github.repository_owner == 'nvidia' }}
82+
permissions:
83+
contents: read # This is required for actions/checkout
84+
needs:
85+
- ci-vars
86+
- build
87+
secrets: inherit
88+
uses: ./.github/workflows/test-wheel-windows.yml
89+
with:
90+
build-type: pull-request
91+
host-platform: ${{ matrix.host-platform }}
92+
build-ctk-ver: ${{ needs.ci-vars.outputs.CUDA_BUILD_VER }}
9493

9594
doc:
9695
name: Docs
@@ -114,8 +113,8 @@ jobs:
114113
checks: read
115114
needs:
116115
- build
117-
# - test-linux # Temporarily disabled for documentation theme migration
118-
# - test-windows # Temporarily disabled for documentation theme migration
116+
- test-linux
117+
- test-windows
119118
- doc
120119
secrets: inherit
121120
uses: ./.github/workflows/status-check.yml

0 commit comments

Comments
 (0)