Skip to content

Commit

Permalink
gha: Fix feature test artifact upload
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
sayboras committed Jan 24, 2025
1 parent 981e9e1 commit 5751a66
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions .github/workflows/tests-e2e-upgrade.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -563,11 +563,7 @@ jobs:
cat <<EOF > 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
Expand Down

0 comments on commit 5751a66

Please sign in to comment.