Skip to content

Commit 3b0da86

Browse files
committed
ci(common): rename jobs to make it easier checks requirements
Adding a convention that a job name with `(bpr)` will be a branch protection requirement (to make it easier to find them in Github UI)
1 parent 89fa3e8 commit 3b0da86

34 files changed

+107
-57
lines changed

.github/workflows/charts-helm-checks.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ env:
88

99
jobs:
1010
check-changes:
11+
name: charts-helm-checks/check-changes
1112
permissions:
1213
actions: 'read'
1314
contents: 'read'
@@ -24,7 +25,8 @@ jobs:
2425
charts:
2526
- '.github/workflows/charts-helm-checks.yml'
2627
- 'charts/**'
27-
helm-charts-lint:
28+
lint:
29+
name: charts-helm-checks/lint
2830
needs: check-changes
2931
if: ${{ needs.check-changes.outputs.changes-charts == 'true' }}
3032
runs-on: 'ubuntu-latest'
@@ -36,7 +38,8 @@ jobs:
3638
uses: WyriHaximus/github-action-helm3@fc4ba26e75cf5d08182c6ce3b72623c8bfd7272b # v3.1.0
3739
with:
3840
exec: helm lint charts/*
39-
helm-charts-test:
41+
test:
42+
name: charts-helm-checks/test (bpr)
4043
needs: check-changes
4144
if: ${{ needs.check-changes.outputs.changes-charts == 'true' }}
4245
runs-on: ubuntu-latest

.github/workflows/charts-helm-release.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ env:
1414
HELM_VERSION: v3.16.4
1515
jobs:
1616
check-changes:
17+
name: charts-helm-release/check-changes
1718
permissions:
1819
actions: 'read'
1920
contents: 'read'
@@ -30,8 +31,9 @@ jobs:
3031
charts:
3132
- '.github/workflows/charts-helm-release.yml'
3233
- 'charts/**'
33-
helm-charts-release:
34+
release:
3435
needs: check-changes
36+
name: charts-helm-release/release
3537
if: ${{ needs.check-changes.outputs.changes-charts == 'true' }}
3638
runs-on: ubuntu-latest
3739
environment: main

.github/workflows/common-pull-request-lint.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ env:
88

99
jobs:
1010
lint:
11+
name: common-pull-request/lint (bpr)
1112
runs-on: ubuntu-latest
1213
steps:
1314
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2

.github/workflows/coprocessor-benchmark-cpu.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ env:
2828

2929
jobs:
3030
setup-instance:
31-
name: Setup instance (fhevm-coprocessor-benchmarks-cpu)
31+
name: coprocessor-benchmarks-cpu/setup-instance
3232
runs-on: ubuntu-latest
3333
permissions:
3434
contents: read
@@ -46,8 +46,8 @@ jobs:
4646
backend: aws
4747
profile: bench
4848

49-
fhevm-coprocessor-benchmarks-cpu:
50-
name: Coprocessor CPU benchmarks (hpc7a.96xlarge)
49+
benchmarks-cpu:
50+
name: coprocessor-benchmarks-cpu/benchmarks-cpu (bpr)
5151
needs: setup-instance
5252
runs-on: ${{ needs.setup-instance.outputs.runner-name }}
5353
continue-on-error: true
@@ -174,9 +174,9 @@ jobs:
174174
--slab-url "${{ secrets.SLAB_URL }}"
175175
176176
teardown-instance:
177-
name: Teardown instance (fhevm-coprocessor-benchmarks-cpu)
177+
name: coprocessor-benchmarks-cpu/teardown
178178
if: ${{ always() && needs.setup-instance.result == 'success' }}
179-
needs: [ setup-instance, fhevm-coprocessor-benchmarks-cpu ]
179+
needs: [ setup-instance, benchmarks-cpu ]
180180
runs-on: ubuntu-latest
181181
permissions:
182182
contents: read

.github/workflows/coprocessor-benchmark-gpu.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ env:
4141

4242
jobs:
4343
parse-inputs:
44+
name: coprocessor-benchmark-gpu/parse-inputs
4445
runs-on: ubuntu-latest
4546
permissions:
4647
contents: read
@@ -59,7 +60,7 @@ jobs:
5960
echo "name=$(echo '${{ inputs.profile }}' | sed 's|.*[[:space:]](\(.*\))|\1|')" >> "${GITHUB_OUTPUT}"
6061
6162
setup-instance:
62-
name: Setup instance (fhevm-coprocessor-benchmarks-gpu)
63+
name: coprocessor-benchmark-gpu/setup-instance
6364
needs: parse-inputs
6465
runs-on: ubuntu-latest
6566
permissions:
@@ -78,8 +79,8 @@ jobs:
7879
backend: hyperstack
7980
profile: ${{ needs.parse-inputs.outputs.profile }}
8081

81-
fhevm-coprocessor-benchmarks-gpu:
82-
name: Coprocessor GPU benchmarks on (${{ needs.parse-inputs.outputs.profile }})
82+
benchmark:
83+
name: coprocessor-benchmark-gpu/benchmark-gpu (bpr)
8384
needs: [ parse-inputs, setup-instance ]
8485
runs-on: ${{ needs.setup-instance.outputs.runner-name }}
8586
continue-on-error: true
@@ -210,9 +211,9 @@ jobs:
210211
--slab-url "${{ secrets.SLAB_URL }}"
211212
212213
teardown-instance:
213-
name: Teardown instance (fhevm-backend-gpu L40 test)
214+
name: coprocessor-benchmark-gpu/teardown
214215
if: ${{ always() && needs.setup-instance.result == 'success' }}
215-
needs: [ setup-instance, fhevm-coprocessor-benchmarks-gpu ]
216+
needs: [ setup-instance, benchmark ]
216217
runs-on: ubuntu-latest
217218
permissions:
218219
contents: read

.github/workflows/coprocessor-cargo-tests.yml

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

1010
jobs:
1111
check-changes:
12+
name: coprocessor-cargo-test/check-changes
1213
permissions:
1314
actions: 'read'
1415
contents: 'read'
@@ -28,6 +29,7 @@ jobs:
2829
- coprocessor/proto/**
2930
- host-contracts/**
3031
cargo-tests:
32+
name: coprocessor-cargo-test/cargo-tests (bpr)
3133
needs: check-changes
3234
if: ${{ needs.check-changes.outputs.changes-rust-files == 'true' }}
3335
permissions:

.github/workflows/coprocessor-dependency-analysis.yml

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

1010
jobs:
1111
check-changes:
12+
name: coprocessor-dependency-analysis/check-changes
1213
permissions:
1314
actions: 'read'
1415
contents: 'read'
@@ -26,6 +27,7 @@ jobs:
2627
- .github/workflows/coprocessor-dependency-analysis.yml
2728
- coprocessor/fhevm-engine/**
2829
dependencies-check:
30+
name: coprocessor-dependency-analysis/dependencies-check (bpr)
2931
needs: check-changes
3032
if: ${{ needs.check-changes.outputs.changes-rust-files == 'true' }}
3133
runs-on: ubuntu-latest

.github/workflows/coprocessor-docker-build-db-migration.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ concurrency:
1515

1616
jobs:
1717
check-changes:
18+
name: coprocessor-docker-build-db-migration/check-changes
1819
permissions:
1920
actions: 'read'
2021
contents: 'read'
@@ -32,6 +33,7 @@ jobs:
3233
- .github/workflows/coprocessor-docker-build-db-migration.yml
3334
- coprocessor/fhevm-engine/db-migration/**
3435
docker-db-migration:
36+
name: coprocessor-docker-build-db-migration/docker-db-migration
3537
needs: check-changes
3638
if: ${{ needs.check-changes.outputs.changes-db-migration-files == 'true' || github.event_name == 'release' }}
3739
uses: zama-ai/ci-templates/.github/workflows/docker_common.yml@44333c96991d6747e0bef6a3308bfd98b20390f8 # main
@@ -49,4 +51,4 @@ jobs:
4951
image-name: "fhevm/coprocessor/db-migration"
5052
generate-dev-image: false
5153
docker-file: "fhevm-engine/db-migration/Dockerfile"
52-
arm-build: true
54+
arm-build: true

.github/workflows/coprocessor-docker-build-gw-listener.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ concurrency:
1515

1616
jobs:
1717
check-changes:
18+
name: coprocessor-docker-build-gw-listener/check-changes
1819
permissions:
1920
actions: 'read'
2021
contents: 'read'
@@ -31,7 +32,8 @@ jobs:
3132
coprocessor-gw-listener:
3233
- .github/workflows/coprocessor-docker-build-gw-listener.yml
3334
- coprocessor/fhevm-engine/gw-listener/**
34-
docker-fhevm-coprocessor:
35+
build:
36+
name: coprocessor-docker-build-gw-listener/build (bpr)
3537
needs: check-changes
3638
if: ${{ needs.check-changes.outputs.changes-coprocessor-gw-listener == 'true' || github.event_name == 'release' }}
3739
uses: zama-ai/ci-templates/.github/workflows/common-docker.yml@ef9d90453e0d776453026c751fbbd48e410d605a
@@ -52,4 +54,4 @@ jobs:
5254
push_image: true
5355
image-name: "fhevm/coprocessor/gw-listener"
5456
docker-file: "./coprocessor/fhevm-engine/gw-listener/Dockerfile"
55-
app-cache-dir: 'fhevm-coprocessor-gw-listener'
57+
app-cache-dir: 'fhevm-coprocessor-gw-listener'

.github/workflows/coprocessor-docker-build.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ concurrency:
1515

1616
jobs:
1717
check-changes:
18+
name: coprocessor-docker-build/check-changes
1819
permissions:
1920
actions: 'read'
2021
contents: 'read'
@@ -33,7 +34,8 @@ jobs:
3334
- coprocessor/fhevm-engine/**
3435
- coprocessor/proto/**
3536
- host-contracts/**
36-
docker-fhevm-coprocessor:
37+
build:
38+
name: coprocessor-docker-build/build (bpr)
3739
needs: check-changes
3840
if: ${{ needs.check-changes.outputs.changes-coprocessor-files == 'true' || github.event_name == 'release' }}
3941
uses: zama-ai/ci-templates/.github/workflows/docker_common.yml@44333c96991d6747e0bef6a3308bfd98b20390f8 # main
@@ -51,4 +53,4 @@ jobs:
5153
image-name: "fhevm/coprocessor"
5254
generate-dev-image: false
5355
docker-file: "coprocessor/fhevm-engine/coprocessor/Dockerfile"
54-
arm-build: true
56+
arm-build: true

0 commit comments

Comments
 (0)