Skip to content

Commit 0021bc5

Browse files
authored
[ARCH-333] Update keystore github workflow settings (#1685)
1 parent 023dd1d commit 0021bc5

File tree

1 file changed

+42
-6
lines changed

1 file changed

+42
-6
lines changed

.github/workflows/keystore.yml

Lines changed: 42 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,34 @@ name: Keystore Checks
22
permissions:
33
contents: read
44

5-
on:
6-
push:
7-
paths:
8-
- "keystore/**"
5+
on: push
96

107
jobs:
8+
changes:
9+
name: detect changes
10+
runs-on: ubuntu-latest
11+
outputs:
12+
keystore-src: ${{ steps.keystore-changes.outputs.src }}
13+
steps:
14+
- name: Checkout the repo
15+
uses: actions/checkout@v4
16+
with:
17+
persist-credentials: false
18+
- uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2
19+
id: keystore-changes
20+
with:
21+
filters: |
22+
src:
23+
- 'keystore/**'
24+
1125
run-tests:
26+
name: run tests
27+
runs-on: ubuntu-latest
28+
needs: changes
29+
if: needs.changes.outputs.keystore-src == 'true'
1230
defaults:
1331
run:
1432
working-directory: keystore
15-
runs-on: ubuntu-latest
1633
steps:
1734
- name: Checkout
1835
uses: actions/checkout@v4
@@ -30,7 +47,13 @@ jobs:
3047
run: go test ./... -coverpkg=./... -coverprofile=coverage.txt
3148

3249
build-race-tests:
50+
name: race tests
3351
runs-on: ubuntu-latest
52+
needs: changes
53+
if: needs.changes.outputs.keystore-src == 'true'
54+
defaults:
55+
run:
56+
working-directory: keystore
3457
steps:
3558
- name: Checkout
3659
uses: actions/checkout@v4
@@ -61,4 +84,17 @@ jobs:
6184
with:
6285
name: go-race-results
6386
path: |
64-
./race.*
87+
./race.*
88+
89+
gate:
90+
name: summary gate
91+
runs-on: ubuntu-latest
92+
needs: [changes, run-tests, build-race-tests]
93+
if: always()
94+
steps:
95+
- name: Fail if any job ran and failed
96+
if: needs.changes.outputs.keystore-src == 'true' &&
97+
(needs.run-tests.outcome != 'success' ||
98+
needs.build-race-tests.outcome != 'success')
99+
run: exit 1
100+

0 commit comments

Comments
 (0)