Skip to content

Commit cd43efc

Browse files
committed
pipe scenario failure correctly
1 parent 53d8f13 commit cd43efc

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

.github/workflows/nightly-throughput-stress.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,11 @@ jobs:
118118
- name: Run throughput stress scenario with local SDK
119119
working-directory: omes
120120
run: |
121+
# This makes the pipeline return the exit code of the first failing command
122+
# Otherwise the output of the `tee` command will be used
123+
# (which is troublesome when the scenario fails but the `tee` command succeeds)
124+
set -o pipefail
125+
121126
# Use run-scenario-with-worker to build and run in one step
122127
# Pass the SDK directory as --version for local testing
123128
# Note: The hardcoded values below match OMES defaults, except:
@@ -175,4 +180,5 @@ jobs:
175180
else
176181
echo "❌ Throughput stress test failed"
177182
echo "📦 Check the uploaded artifacts for detailed logs and state"
183+
exit 1
178184
fi

0 commit comments

Comments
 (0)