Skip to content

Commit f83e1b0

Browse files
committed
ci: Use bigger runner for icx job variation, needs more disk space (#2047)
A week or two ago, the icx job variation stopped working, not enough space to finish installing the packages containing icx and related material. We didn't change, the Intel package (pegged to a specific version from a couple years ago) didn't change, the ASWF docker container (also old) didn't change, so we're assuming that the GHA runner itself changed, leaving us less disk space, which is now not enough. The easiest way out of this for now is for the icx job variation to use a paid bigger GHA runner with more "disk space". This means it can only be enabled when running on the main repo, which uses the ASWF's account that has paid runners. Sorry, CI can't currently test with icx you people's individual forks until we figure out a way to require less space. Until then, developers who don't have icx locally will need to submit the PR to see how it does on icx. A little bit of workflow magic was needed to ensure that this job variation doesn't run at all on a developer's fork -- we don't want it to run, and look like a failed test just because they don't have an account that doesn't allow use of the big paid runners. Signed-off-by: Larry Gritz <[email protected]>
1 parent 1f7c6fb commit f83e1b0

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

.github/workflows/build-steps.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ on:
1212
workflow_call:
1313
# This inputs receive values via the "with:" section in ci_workflow.yml
1414
inputs:
15+
disable:
16+
type: boolean
17+
default: false
1518
build:
1619
type: string
1720
runner:
@@ -85,6 +88,7 @@ jobs:
8588

8689
steps:
8790
name: "${{inputs.cxx_compiler}} c++${{inputs.cxx_std}} py${{inputs.python_ver}}"
91+
if: ${{ !inputs.disable }}
8892
runs-on: ${{ inputs.runner }}
8993
container:
9094
image: ${{ inputs.container }}

.github/workflows/ci.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ name: CI
99

1010
on:
1111
push:
12-
# Skip jobs when only documentation files are changed
1312
# Skip jobs when only documentation files are changed
1413
paths:
1514
- '**'
@@ -214,6 +213,7 @@ jobs:
214213
uses: ./.github/workflows/build-steps.yml
215214
with:
216215
nametag: ${{ matrix.nametag || 'unnamed!' }}
216+
disable: ${{ matrix.disable || false }}
217217
runner: ${{ matrix.runner || 'ubuntu-latest' }}
218218
container: ${{ matrix.container }}
219219
cc_compiler: ${{ matrix.cc_compiler || 'gcc' }}
@@ -381,7 +381,13 @@ jobs:
381381
OPENCOLORIO_CMAKE_FLAGS="-DCMAKE_CXX_COMPILER=g++"
382382
- desc: icx/C++17 llvm14 py3.10 oiio-3.0 avx2
383383
nametag: linux-icx
384-
runner: ubuntu-latest
384+
# This job variation needs a bigger runner with more "disk space",
385+
# so only enable it when running on the main repo. Sorry, CI can't
386+
# currently test with icx you people's individual forks until we
387+
# figure out a way to require less space. Until then, they'll need
388+
# to submit the PR to see how it does on icx.
389+
runner: ${{ (github.repository_owner == 'AcademySoftwareFoundation' && 'ubuntu-22.04-8c-32g-300h') || 'ubuntu-latest' }}
390+
disable: ${{ github.repository_owner != 'AcademySoftwareFoundation' }}
385391
container: aswftesting/ci-osl:2023-clang15
386392
cc_compiler: icx
387393
cxx_compiler: icpx

0 commit comments

Comments
 (0)