Skip to content

Commit abed32e

Browse files
committed
simplify workflow
1 parent 88ef676 commit abed32e

File tree

1 file changed

+7
-11
lines changed

1 file changed

+7
-11
lines changed

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

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ jobs:
117117
118118
- name: Run throughput stress scenario with local SDK
119119
working-directory: omes
120-
continue-on-error: true
121120
run: |
122121
# Use run-scenario-with-worker to build and run in one step
123122
# Pass the SDK directory as --version for local testing
@@ -139,18 +138,16 @@ jobs:
139138
--option min-throughput-per-hour=1000 \
140139
2>&1 | tee $WORKER_LOG_DIR/scenario.log
141140
142-
echo "SCENARIO_EXIT_CODE=${PIPESTATUS[0]}" >> $GITHUB_ENV
143-
144141
- name: Upload logs on failure
145-
if: env.SCENARIO_EXIT_CODE != '0'
142+
if: failure() || cancelled()
146143
uses: actions/upload-artifact@v4
147144
with:
148145
name: throughput-stress-logs
149146
path: ${{ env.WORKER_LOG_DIR }}
150147
retention-days: 30
151148

152149
- name: Notify Slack on failure
153-
if: env.SCENARIO_EXIT_CODE != '0'
150+
if: failure() || cancelled()
154151
uses: slackapi/slack-github-action@v2
155152
with:
156153
webhook-type: incoming-webhook
@@ -170,13 +167,12 @@ jobs:
170167
env:
171168
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_SDK_ALERTS_WEBHOOK }}
172169

173-
- name: Fail if scenario failed
170+
- name: Report test results
174171
if: always()
175172
run: |
176-
if [ "${SCENARIO_EXIT_CODE:-1}" != "0" ]; then
177-
echo "❌ Throughput stress test failed with exit code ${SCENARIO_EXIT_CODE}"
178-
echo "Check the artifacts for detailed logs and state"
179-
exit 1
180-
else
173+
if [ "${{ steps.*.outcome }}" = "success" ]; then
181174
echo "✅ Throughput stress test completed successfully"
175+
else
176+
echo "❌ Throughput stress test failed"
177+
echo "📦 Check the uploaded artifacts for detailed logs and state"
182178
fi

0 commit comments

Comments
 (0)