From 5751a66faaa6b1cb5d2ea97f181333f455cc7388 Mon Sep 17 00:00:00 2001 From: Tam Mach Date: Fri, 24 Jan 2025 21:21:16 +1100 Subject: [PATCH] gha: Fix feature test artifact upload This is to fix make sure the features tested file artifact is upload, two changes are done in this commit: - Remove `find /host/test/test_results` block, as this is mainly for ginkgo tests but not connectivity tests. - Add prefix `feature-status-` for json output. Sample failure: https://github.com/cilium/cilium/actions/runs/12941992870 Signed-off-by: Tam Mach --- .github/workflows/tests-e2e-upgrade.yaml | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/.github/workflows/tests-e2e-upgrade.yaml b/.github/workflows/tests-e2e-upgrade.yaml index 296d00a476514..1df9c30d3f235 100644 --- a/.github/workflows/tests-e2e-upgrade.yaml +++ b/.github/workflows/tests-e2e-upgrade.yaml @@ -563,11 +563,7 @@ jobs: cat < run-in-lvm.sh #!/usr/bin/env bash ./cilium-cli features status -o markdown --output-file="feature-status.md" - ./cilium-cli features status -o json --output-file="${1}.json" - cd /host - if ! find /host/test/test_results -type f -iname 'feature-status-*.json' | grep -q .; then - exit 0 - fi + ./cilium-cli features status -o json --output-file="feature-status-${1}.json" find . -type f -iname 'feature-status-*.json' | while IFS= read -r file; do mv "\$file" "./${{ env.job_name }}-\${file##*/feature-status-}" done