Skip to content

Commit 88156c3

Browse files
authored
Finish setup on error (#5419)
* Fast-fail on errors * Fix an indentation error * Fail the setup script on an error
1 parent 6c47e60 commit 88156c3

File tree

8 files changed

+35
-12
lines changed

8 files changed

+35
-12
lines changed

.github/workflows/0-on-demand.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,9 @@ env:
6464
jobs:
6565
openwhisk:
6666
runs-on: ubuntu-22.04
67+
continue-on-error: false
68+
strategy:
69+
fail-fast: true
6770
steps:
6871
- name: Checkout
6972
uses: actions/checkout@v3

.github/workflows/1-unit.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,14 @@ env:
4949

5050
# github
5151
GH_BUILD: ${{ github.event_name }}-${{ github.sha }}
52-
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
52+
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
5353

5454
jobs:
5555
openwhisk:
5656
runs-on: ubuntu-22.04
57+
continue-on-error: false
58+
strategy:
59+
fail-fast: true
5760
steps:
5861
- name: Checkout
5962
uses: actions/checkout@v3

.github/workflows/2-system.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,14 @@ env:
4444

4545
# github
4646
GH_BUILD: ${{ github.event_name }}-${{ github.sha }}
47-
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
47+
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
4848

4949
jobs:
5050
openwhisk:
5151
runs-on: ubuntu-22.04
52+
continue-on-error: false
53+
strategy:
54+
fail-fast: true
5255
steps:
5356
- name: Checkout
5457
uses: actions/checkout@v3

.github/workflows/3-multi-runtime.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,14 @@ env:
4444

4545
# github
4646
GH_BUILD: ${{ github.event_name }}-${{ github.sha }}
47-
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
47+
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
4848

4949
jobs:
5050
openwhisk:
5151
runs-on: ubuntu-22.04
52+
continue-on-error: false
53+
strategy:
54+
fail-fast: true
5255
steps:
5356
- name: Checkout
5457
uses: actions/checkout@v3

.github/workflows/4-standalone.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,14 @@ env:
4444

4545
# github
4646
GH_BUILD: ${{ github.event_name }}-${{ github.sha }}
47-
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
47+
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
4848

4949
jobs:
5050
openwhisk:
5151
runs-on: ubuntu-22.04
52+
continue-on-error: false
53+
strategy:
54+
fail-fast: true
5255
steps:
5356
- name: Checkout
5457
uses: actions/checkout@v3

.github/workflows/5-scheduler.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,14 @@ env:
4444

4545
# github
4646
GH_BUILD: ${{ github.event_name }}-${{ github.sha }}
47-
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
47+
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
4848

4949
jobs:
5050
openwhisk:
5151
runs-on: ubuntu-22.04
52+
continue-on-error: false
53+
strategy:
54+
fail-fast: true
5255
steps:
5356
- name: Checkout
5457
uses: actions/checkout@v3

.github/workflows/6-performance.yaml

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -44,11 +44,14 @@ env:
4444

4545
# github
4646
GH_BUILD: ${{ github.event_name }}-${{ github.sha }}
47-
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
47+
GH_BRANCH: ${{ github.head_ref || github.ref_name }}
4848

4949
jobs:
5050
openwhisk:
5151
runs-on: ubuntu-22.04
52+
continue-on-error: false
53+
strategy:
54+
fail-fast: true
5255
steps:
5356
- name: Checkout
5457
uses: actions/checkout@v3
@@ -79,9 +82,9 @@ jobs:
7982
- run: OPENWHISK_HOST="172.17.0.1" USERS="1" REQUESTS_PER_SEC="1" ./gradlew gatlingRun-org.apache.openwhisk.ColdBlockingInvokeSimulation
8083
continue-on-error: true
8184
- name: Slack Notification
82-
run: >
83-
./tools/github/writeOnSlack.sh
84-
"[$TEST_SUITE]" ${{ steps.tests.outcome }} on ${GH_BUILD}
85-
$'\nbranch:' $GH_BRANCH
86-
$'\nmessage:' "$(git log -1 --oneline | cat)"
87-
$'\nCheck GitHub logs for results'
85+
run: >
86+
./tools/github/writeOnSlack.sh
87+
"[$TEST_SUITE]" ${{ steps.tests.outcome }} on ${GH_BUILD}
88+
$'\nbranch:' $GH_BRANCH
89+
$'\nmessage:' "$(git log -1 --oneline | cat)"
90+
$'\nCheck GitHub logs for results'

tools/github/setup.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@
2020
#then echo skipping setup ; exit 0
2121
#fi
2222

23+
set -e
24+
2325
# retries a command for five times and exits with the non-zero exit if even after
2426
# the retries the command did not succeed.
2527
function retry() {

0 commit comments

Comments
 (0)