Skip to content

Commit

Permalink
GitHub Actions: use report_name instead of variables in publish-logs (#…
Browse files Browse the repository at this point in the history
…897)

Previous implementation did not use test suite name in publishing logs,
therefore for 11 different workflows we got only 4 artifacts, new implimentation
uses report_name parameter in publish-logs action which makes it more flexible to
set up any report name user wants and makes it more clear how it forms in
`run-eden-test` action. And, of course, it adds test suite name now

Signed-off-by: Pavel Abramov <[email protected]>
  • Loading branch information
uncleDecart authored Oct 4, 2023
1 parent b8d8adb commit 0cd3006
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
7 changes: 2 additions & 5 deletions .github/actions/publish-logs/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,9 @@ name: 'Collect, count and publish logs'
description: 'Collect, process and publish logs from Eden'

inputs:
file_system:
report_name:
required: true
type: string
tpm_enabled:
required: true
type: bool

runs:
using: 'composite'
Expand Down Expand Up @@ -48,7 +45,7 @@ runs:
if: ${{ always() }}
uses: actions/upload-artifact@v3
with:
name: eden-report-tpm-${{ inputs.tpm_enabled }}-${{ inputs.file_system }}
name: ${{ inputs.report_name }}
path: |
${{ github.workspace }}/eden/eve-info.tar.gz # created by collect-info action
${{ github.workspace }}/eden/trace.log
Expand Down
3 changes: 1 addition & 2 deletions .github/actions/run-eden-test/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,7 @@ runs:
if: always()
uses: ./eden/.github/actions/publish-logs
with:
file_system: ${{ inputs.file_system }}
tpm_enabled: ${{ inputs.tpm_enabled }}
report_name: eden-report-${{ inputs.suite }}-tpm-${{ inputs.tpm_enabled }}-${{ inputs.file_system }}
- name: Clean up after test
if: always()
run: |
Expand Down

0 comments on commit 0cd3006

Please sign in to comment.