Skip to content

Commit 579ccaf

Browse files
vdemeestertekton-robot
authored andcommitted
ci: replace e2e-only fan-in with unified CI summary job
- Fix docs-only PRs failing the "e2e tests" check because skipped jobs were treated as failures - Consolidate all CI jobs into a single fan-in that accepts both "success" and "skipped" as passing states Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
1 parent c808c3d commit 579ccaf

File tree

1 file changed

+27
-7
lines changed

1 file changed

+27
-7
lines changed

.github/workflows/ci.yaml

Lines changed: 27 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -175,16 +175,36 @@ jobs:
175175
needs: [build]
176176
uses: ./.github/workflows/e2e-matrix.yml
177177

178-
e2e-summary:
179-
name: e2e tests
180-
needs: [e2e-tests]
178+
ci-summary:
179+
name: CI summary
180+
needs: [build, buildFips, linting, tests, generated, multi-arch-build, e2e-tests]
181181
runs-on: ubuntu-latest
182182
if: always()
183183
steps:
184-
- name: Check e2e results
184+
- name: Check CI results
185185
run: |
186-
if [ "${{ needs.e2e-tests.result }}" != "success" ]; then
187-
echo "e2e-tests failed or were cancelled"
186+
results=(
187+
"build=${{ needs.build.result }}"
188+
"buildFips=${{ needs.buildFips.result }}"
189+
"linting=${{ needs.linting.result }}"
190+
"tests=${{ needs.tests.result }}"
191+
"generated=${{ needs.generated.result }}"
192+
"multi-arch-build=${{ needs.multi-arch-build.result }}"
193+
"e2e-tests=${{ needs.e2e-tests.result }}"
194+
)
195+
failed=0
196+
for r in "${results[@]}"; do
197+
name="${r%%=*}"
198+
result="${r#*=}"
199+
echo "${name}: ${result}"
200+
if [ "$result" != "success" ] && [ "$result" != "skipped" ]; then
201+
failed=1
202+
fi
203+
done
204+
if [ "$failed" -eq 1 ]; then
205+
echo ""
206+
echo "Some CI jobs failed or were cancelled"
188207
exit 1
189208
fi
190-
echo "All e2e tests passed"
209+
echo ""
210+
echo "All CI checks passed"

0 commit comments

Comments
 (0)