Skip to content

Commit 2ebca7b

Browse files
authored
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 4b5fc37 commit 2ebca7b

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
- '**'
@@ -200,6 +199,7 @@ jobs:
200199
uses: ./.github/workflows/build-steps.yml
201200
with:
202201
nametag: ${{ matrix.nametag || 'unnamed!' }}
202+
disable: ${{ matrix.disable || false }}
203203
runner: ${{ matrix.runner || 'ubuntu-latest' }}
204204
container: ${{ matrix.container }}
205205
cc_compiler: ${{ matrix.cc_compiler || 'gcc' }}
@@ -367,7 +367,13 @@ jobs:
367367
OPENCOLORIO_CMAKE_FLAGS="-DCMAKE_CXX_COMPILER=g++"
368368
- desc: icx/C++17 llvm14 py3.10 oiio-3.0 avx2
369369
nametag: linux-icx
370-
runner: ubuntu-latest
370+
# This job variation needs a bigger runner with more "disk space",
371+
# so only enable it when running on the main repo. Sorry, CI can't
372+
# currently test with icx you people's individual forks until we
373+
# figure out a way to require less space. Until then, they'll need
374+
# to submit the PR to see how it does on icx.
375+
runner: ${{ (github.repository_owner == 'AcademySoftwareFoundation' && 'ubuntu-22.04-8c-32g-300h') || 'ubuntu-latest' }}
376+
disable: ${{ github.repository_owner != 'AcademySoftwareFoundation' }}
371377
container: aswftesting/ci-osl:2023-clang15
372378
cc_compiler: icx
373379
cxx_compiler: icpx

0 commit comments

Comments
 (0)