Skip to content

Commit deaee71

Browse files
committed
.github/actions: only upload files with features-tested prefix
When uploading the feature .json files, we should only upload the ones with this prefix. Signed-off-by: André Martins <andre@cilium.io>
1 parent 2f0652b commit deaee71

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

.github/actions/feature-status/action.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ runs:
2121
shell: bash
2222
run: |
2323
${{ inputs.cilium-cli }} features status -o markdown --output-file="feature-status.md"
24-
${{ inputs.cilium-cli }} features status -o json --output-file="${{ inputs.json-filename }}.json"
24+
${{ inputs.cilium-cli }} features status -o json --output-file="features-tested-${{ inputs.json-filename }}.json"
2525
2626
- name: Report summary for features enabled on the agent
2727
if: ${{ always() }}

.github/actions/post-logic/action.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ runs:
2828
uses: ./.github/actions/feature-status
2929
with:
3030
title: "Summary of all features tested"
31-
json-filename: "features-tested-${{ inputs.artifacts_suffix }}"
31+
json-filename: "${{ inputs.artifacts_suffix }}"
3232

3333
- name: Post-test information gathering
3434
if: ${{ always() && ( ( inputs.job_status == 'failure' && inputs.capture_sysdump == 'true') || inputs.always_capture_sysdump == 'true') }}
@@ -61,7 +61,7 @@ runs:
6161
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
6262
with:
6363
name: "features-tested-${{ inputs.artifacts_suffix }}"
64-
path: ./*.json
64+
path: ./features-tested-*.json
6565

6666
- name: Publish Test Results As GitHub Summary
6767
if: ${{ always() }}

.github/workflows/conformance-runtime.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,7 @@ jobs:
467467
docker create --name cilium-dbg quay.io/${{ env.QUAY_ORGANIZATION_DEV }}/cilium-ci:${{ steps.vars.outputs.sha }}
468468
docker cp cilium-dbg:/usr/bin/cilium-dbg /usr/local/bin/cilium-dbg
469469
docker rm cilium-dbg
470-
cilium-dbg metrics list -p cilium_feature -o json > '/host/${{ env.job_name }} (${{ matrix.focus }}).json'
470+
cilium-dbg metrics list -p cilium_feature -o json > '/host/features-tested-${{ env.job_name }} (${{ matrix.focus }}).json'
471471
472472
- name: Debug failure on VM
473473
# Only debug the failure on the LVH that have Cilium running as a service,
@@ -524,7 +524,7 @@ jobs:
524524
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
525525
with:
526526
name: features-tested-${{ matrix.focus }}
527-
path: ${{ env.job_name }}*.json
527+
path: features-tested-${{ env.job_name }}*.json
528528

529529
- name: Publish Test Results As GitHub Summary
530530
if: ${{ always() && runner.arch != 'ARM64' }}

0 commit comments

Comments
 (0)