Skip to content

Commit

Permalink
Merge pull request #1193 from zama-ai/youben11-patch-8
Browse files Browse the repository at this point in the history
chore(ci): rename steps to more meaningful names
  • Loading branch information
youben11 authored Jan 10, 2025
2 parents 43150a5 + e5c7db8 commit 57eef1c
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/concrete_compiler_test_cpu_distributed.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ concurrency:
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}

jobs:
setup-instance:
start-instance:
runs-on: ubuntu-latest
steps:
- name: Start instance
Expand All @@ -38,8 +38,8 @@ jobs:

build-and-run-test:
# The distributed-ci runner is registered on the instance configured in the slurm-cluster profile.
# It's why we need to setup-instance
needs: setup-instance
# It's why we need to start-instance
needs: start-instance
runs-on: distributed-ci
steps:
- name: Instance cleanup
Expand Down Expand Up @@ -83,9 +83,9 @@ jobs:
SLACK_COLOR: ${{ job.status }}
SLACK_MESSAGE: "build-and-run-test finished with status: ${{ job.status }}. (${{ env.ACTION_RUN_URL }})"

teardown-instance:
needs: [ setup-instance, build-and-run-test ]
if: ${{ always() && needs.setup-instance.result != 'skipped' }}
stop-instance:
needs: [ start-instance, build-and-run-test ]
if: ${{ always() && needs.start-instance.result != 'skipped' }}
runs-on: ubuntu-latest
steps:
- name: Stop instance
Expand All @@ -104,4 +104,4 @@ jobs:
uses: rtCamp/action-slack-notify@c33737706dea87cd7784c687dadc9adf1be59990
env:
SLACK_COLOR: ${{ job.status }}
SLACK_MESSAGE: "Instance teardown finished with status: ${{ job.status }}. (${{ env.ACTION_RUN_URL }})"
SLACK_MESSAGE: "Stopping instance finished with status: ${{ job.status }}. (${{ env.ACTION_RUN_URL }})"

0 comments on commit 57eef1c

Please sign in to comment.