Skip to content

Commit fb81f56

Browse files
committed
feat(ci): add e2e test validation to release workflow
Add test-e2e job that calls run-e2e-test.yml before building release artifacts. Release fails if e2e tests fail. Signed-off-by: Matthew F Leader <mleader@redhat.com>
1 parent 87c7216 commit fb81f56

File tree

2 files changed

+27
-0
lines changed

2 files changed

+27
-0
lines changed

.github/workflows/generate-release.yml

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,20 @@ env:
2121
GIT_USER_EMAIL: github-actions[bot]@users.noreply.github.com
2222

2323
jobs:
24+
test-e2e:
25+
uses: ./.github/workflows/run-e2e-test.yml
26+
2427
generate-release:
2528
runs-on: ubuntu-24.04
29+
needs: test-e2e
30+
if: always()
2631
outputs:
2732
operator_version: ${{ steps.validate.outputs.operator_version }}
2833
llamastack_version: ${{ steps.validate.outputs.llamastack_version }}
2934
release_branch: ${{ steps.validate.outputs.release_branch }}
35+
e2e_success: ${{ needs.test-e2e.outputs.success }}
36+
e2e_summary: ${{ needs.test-e2e.outputs.summary }}
37+
e2e_run_url: ${{ needs.test-e2e.outputs.run_url }}
3038
steps:
3139
- name: Checkout code
3240
uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3
@@ -157,6 +165,21 @@ jobs:
157165
run: |
158166
make test
159167
168+
- name: Check E2E Test Results
169+
run: |
170+
echo "E2E Test Results:"
171+
echo "Success: ${{ needs.test-e2e.outputs.success }}"
172+
echo "Summary: ${{ needs.test-e2e.outputs.summary }}"
173+
echo "Run URL: ${{ needs.test-e2e.outputs.run_url }}"
174+
175+
if [ "${{ needs.test-e2e.outputs.success }}" != "true" ]; then
176+
echo "E2E tests failed - release cannot proceed"
177+
echo "Check detailed logs: ${{ needs.test-e2e.outputs.run_url }}"
178+
exit 1
179+
else
180+
echo "E2E tests passed - proceeding with release"
181+
fi
182+
160183
- name: Validate build release image
161184
shell: bash
162185
run: |

tests/e2e/e2e_test.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@ import (
99

1010
func TestE2E(t *testing.T) {
1111
registerSchemes()
12+
13+
// TEMPORARY: Force test failure to test release workflow integration
14+
t.Fatal("Testing E2E failure handling in release workflow")
15+
1216
// Run validation tests
1317
t.Run("validation", TestValidationSuite)
1418

0 commit comments

Comments
 (0)