Skip to content

Commit 3b4c6f1

Browse files
committed
Check skip flag in each job step
1 parent dbb7012 commit 3b4c6f1

File tree

10 files changed

+152
-32
lines changed

10 files changed

+152
-32
lines changed

.github/workflows/clang_sanitizers.yml

+12-2
Original file line numberDiff line numberDiff line change
@@ -32,26 +32,30 @@ jobs:
3232
name: Clang UB sanitizer
3333
runs-on: ubuntu-24.04
3434
needs: skip_checks
35-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
35+
if: ${{ github.event.pull_request.draft == false }}
3636
env:
3737
CC: clang
3838
CXX: clang++
3939
# On CI for this test, Ninja is slower than the default:
4040
#CMAKE_GENERATOR: Ninja
4141
steps:
4242
- name: Checkout code
43+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4344
uses: actions/checkout@v4
4445
- name: Install dependencies
46+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4547
run: |
4648
.github/workflows/dependencies/clang.sh 17
4749
- name: CCache Cache
50+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4851
uses: actions/cache@v4
4952
with:
5053
path: ~/.cache/ccache
5154
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
5255
restore-keys: |
5356
ccache-${{ github.workflow }}-${{ github.job }}-git-
5457
- name: Build WarpX
58+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
5559
run: |
5660
export CCACHE_COMPRESS=1
5761
export CCACHE_COMPRESSLEVEL=10
@@ -75,6 +79,7 @@ jobs:
7579
ccache -s
7680
du -hs ~/.cache/ccache
7781
- name: Run with UB sanitizer
82+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
7883
run: |
7984
export OMP_NUM_THREADS=2
8085
#MPI implementations often leak memory
@@ -89,26 +94,30 @@ jobs:
8994
name: Clang thread sanitizer
9095
runs-on: ubuntu-24.04
9196
needs: skip_checks
92-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
97+
if: ${{ github.event.pull_request.draft == false }}
9398
env:
9499
CC: clang
95100
CXX: clang++
96101
# On CI for this test, Ninja is slower than the default:
97102
#CMAKE_GENERATOR: Ninja
98103
steps:
99104
- name: Checkout code
105+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
100106
uses: actions/checkout@v4
101107
- name: Install dependencies
108+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
102109
run: |
103110
.github/workflows/dependencies/clang.sh 17
104111
- name: CCache Cache
112+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
105113
uses: actions/cache@v4
106114
with:
107115
path: ~/.cache/ccache
108116
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
109117
restore-keys: |
110118
ccache-${{ github.workflow }}-${{ github.job }}-git-
111119
- name: Build WarpX
120+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
112121
run: |
113122
export CCACHE_COMPRESS=1
114123
export CCACHE_COMPRESSLEVEL=10
@@ -132,6 +141,7 @@ jobs:
132141
ccache -s
133142
du -hs ~/.cache/ccache
134143
- name: Run with thread sanitizer
144+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
135145
run: |
136146
# Disabling the gds/shmem component by using gds/hash instead
137147
# is required to avoid issues with shared memory.

.github/workflows/codeql.yml

+13-7
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@ jobs:
3030
outputs:
3131
skip: ${{ env.SKIP_CHECKS }}
3232
analyze:
33-
name: Query code
33+
name: Analyze
3434
runs-on: ubuntu-24.04
3535
needs: skip_checks
36-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
36+
if: ${{ github.event.pull_request.draft == false }}
3737
permissions:
3838
actions: read
3939
contents: read
@@ -44,13 +44,15 @@ jobs:
4444
language: [ python, cpp ]
4545
steps:
4646
- name: Checkout code
47+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4748
uses: actions/checkout@v4
4849
- name: Install Python
50+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4951
uses: actions/setup-python@v5
5052
with:
5153
python-version: '3.x'
5254
- name: Install Packages (C++)
53-
if: ${{ matrix.language == 'cpp' }}
55+
if: ${{ needs.skip_checks.outputs.skip == 'false' && matrix.language == 'cpp' }}
5456
run: |
5557
sudo apt-get update
5658
sudo apt-get install --yes cmake openmpi-bin libopenmpi-dev libhdf5-openmpi-dev libadios-openmpi-dev ccache
@@ -60,28 +62,29 @@ jobs:
6062
python -m pip install --upgrade cmake
6163
python -m pipx install cmake
6264
- name: Set Up Cache
63-
if: ${{ matrix.language == 'cpp' }}
65+
if: ${{ needs.skip_checks.outputs.skip == 'false' && matrix.language == 'cpp' }}
6466
uses: actions/cache@v4
6567
with:
6668
path: ~/.cache/ccache
6769
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
6870
restore-keys: |
6971
ccache-${{ github.workflow }}-${{ github.job }}-git-
7072
- name: Configure (C++)
71-
if: ${{ matrix.language == 'cpp' }}
73+
if: ${{ needs.skip_checks.outputs.skip == 'false' && matrix.language == 'cpp' }}
7274
run: |
7375
cmake -S . -B build -DWarpX_OPENPMD=ON
7476
- name: Initialize CodeQL
77+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
7578
uses: github/codeql-action/init@v3
7679
with:
7780
config-file: ./.github/codeql/warpx-codeql.yml
7881
languages: ${{ matrix.language }}
7982
queries: +security-and-quality
8083
- name: Build (py)
81-
if: ${{ matrix.language == 'python' }}
84+
if: ${{ needs.skip_checks.outputs.skip == 'false' && matrix.language == 'python' }}
8285
uses: github/codeql-action/autobuild@v3
8386
- name: Build (C++)
84-
if: ${{ matrix.language == 'cpp' }}
87+
if: ${{ needs.skip_checks.outputs.skip == 'false' && matrix.language == 'cpp' }}
8588
run: |
8689
export CCACHE_COMPRESS=1
8790
export CCACHE_COMPRESSLEVEL=10
@@ -95,12 +98,14 @@ jobs:
9598
export CCACHE_DISABLE=1
9699
cmake --build build -j 4
97100
- name: Perform CodeQL Analysis
101+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
98102
uses: github/codeql-action/analyze@v3
99103
with:
100104
category: "/language:${{ matrix.language }}"
101105
upload: False
102106
output: sarif-results
103107
- name: filter-sarif
108+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
104109
uses: advanced-security/filter-sarif@v1
105110
with:
106111
patterns: |
@@ -114,6 +119,7 @@ jobs:
114119
input: sarif-results/${{ matrix.language }}.sarif
115120
output: sarif-results/${{ matrix.language }}.sarif
116121
- name: Upload SARIF
122+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
117123
uses: github/codeql-action/upload-sarif@v3
118124
with:
119125
sarif_file: sarif-results/${{ matrix.language }}.sarif

.github/workflows/cuda.yml

+17-3
Original file line numberDiff line numberDiff line change
@@ -35,28 +35,33 @@ jobs:
3535
name: NVCC SP
3636
runs-on: ubuntu-22.04
3737
needs: skip_checks
38-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
38+
if: ${{ github.event.pull_request.draft == false }}
3939
env:
4040
CXXFLAGS: "-Werror"
4141
CMAKE_GENERATOR: Ninja
4242
steps:
4343
- name: Checkout code
44+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4445
uses: actions/checkout@v4
4546
- name: Install Python
47+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4648
uses: actions/setup-python@v5
4749
with:
4850
python-version: '3.x'
4951
- name: Install dependencies
52+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
5053
run: |
5154
.github/workflows/dependencies/nvcc.sh 11.7
5255
- name: CCache Cache
56+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
5357
uses: actions/cache@v4
5458
with:
5559
path: ~/.cache/ccache
5660
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
5761
restore-keys: |
5862
ccache-${{ github.workflow }}-${{ github.job }}-git-
5963
- name: Install openPMD-api
64+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
6065
run: |
6166
export CCACHE_COMPRESS=1
6267
export CCACHE_COMPRESSLEVEL=10
@@ -76,6 +81,7 @@ jobs:
7681
-DCMAKE_CXX_COMPILER_LAUNCHER=$(which ccache) \
7782
-DCMAKE_VERBOSE_MAKEFILE=ON
7883
- name: Build WarpX
84+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
7985
run: |
8086
export CCACHE_COMPRESS=1
8187
export CCACHE_COMPRESSLEVEL=10
@@ -110,21 +116,25 @@ jobs:
110116
name: NVCC GNU Make
111117
runs-on: ubuntu-22.04
112118
needs: skip_checks
113-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
119+
if: ${{ github.event.pull_request.draft == false }}
114120
steps:
115121
- name: Checkout code
122+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
116123
uses: actions/checkout@v4
117124
- name: Install dependencies
125+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
118126
run: |
119127
.github/workflows/dependencies/nvcc.sh 11.7
120128
- name: CCache Cache
129+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
121130
uses: actions/cache@v4
122131
with:
123132
path: ~/.cache/ccache
124133
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
125134
restore-keys: |
126135
ccache-${{ github.workflow }}-${{ github.job }}-git-
127136
- name: Build WarpX
137+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
128138
run: |
129139
export CCACHE_COMPRESS=1
130140
export CCACHE_COMPRESSLEVEL=10
@@ -143,23 +153,27 @@ jobs:
143153
name: NVHPC
144154
runs-on: ubuntu-24.04
145155
needs: skip_checks
146-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
156+
if: ${{ github.event.pull_request.draft == false }}
147157
#env:
148158
# # For NVHPC, Ninja is slower than the default:
149159
# CMAKE_GENERATOR: Ninja
150160
steps:
151161
- name: Checkout code
162+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
152163
uses: actions/checkout@v4
153164
- name: Dependencies
165+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
154166
run: .github/workflows/dependencies/nvhpc.sh 25.1
155167
- name: CCache Cache
168+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
156169
uses: actions/cache@v4
157170
with:
158171
path: ~/.cache/ccache
159172
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
160173
restore-keys: |
161174
ccache-${{ github.workflow }}-${{ github.job }}-git-
162175
- name: Build & Install
176+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
163177
run: |
164178
export CCACHE_COMPRESS=1
165179
export CCACHE_COMPRESSLEVEL=10

.github/workflows/hip.yml

+10-2
Original file line numberDiff line numberDiff line change
@@ -34,21 +34,25 @@ jobs:
3434
CXXFLAGS: "-Werror -Wno-deprecated-declarations -Wno-error=pass-failed"
3535
CMAKE_GENERATOR: Ninja
3636
needs: skip_checks
37-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
37+
if: ${{ github.event.pull_request.draft == false }}
3838
steps:
3939
- name: Checkout code
40+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4041
uses: actions/checkout@v4
4142
- name: Install dependencies
43+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4244
shell: bash
4345
run: .github/workflows/dependencies/hip.sh 6.3.2
4446
- name: CCache Cache
47+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
4548
uses: actions/cache@v4
4649
with:
4750
path: ~/.cache/ccache
4851
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
4952
restore-keys: |
5053
ccache-${{ github.workflow }}-${{ github.job }}-git-
5154
- name: Build WarpX
55+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
5256
shell: bash
5357
run: |
5458
export CCACHE_COMPRESS=1
@@ -86,21 +90,25 @@ jobs:
8690
CXXFLAGS: "-Werror -Wno-deprecated-declarations -Wno-error=pass-failed"
8791
CMAKE_GENERATOR: Ninja
8892
needs: skip_checks
89-
if: ${{ github.event.pull_request.draft == false && needs.skip_checks.outputs.skip == 'false' }}
93+
if: ${{ github.event.pull_request.draft == false }}
9094
steps:
9195
- name: Checkout code
96+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
9297
uses: actions/checkout@v4
9398
- name: Install dependencies
99+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
94100
shell: bash
95101
run: .github/workflows/dependencies/hip.sh 6.3.2
96102
- name: CCache Cache
103+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
97104
uses: actions/cache@v4
98105
with:
99106
path: ~/.cache/ccache
100107
key: ccache-${{ github.workflow }}-${{ github.job }}-git-${{ github.sha }}
101108
restore-keys: |
102109
ccache-${{ github.workflow }}-${{ github.job }}-git-
103110
- name: Build WarpX
111+
if: ${{ needs.skip_checks.outputs.skip == 'false' }}
104112
shell: bash
105113
run: |
106114
export CCACHE_COMPRESS=1

0 commit comments

Comments
 (0)