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
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 <uncle.decart@gmail.com>
  • Loading branch information
uncleDecart committed Oct 4, 2023
1 parent 8610724 commit 17725dc
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 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
1 change: 1 addition & 0 deletions .github/actions/run-eden-test/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ runs:
if: always()
uses: ./eden/.github/actions/publish-logs
with:
report_name: eden-report-${{ inputs.suite }}-tpm-${{ inputs.tpm_enabled }}-${{ inputs.file_system }}
file_system: ${{ inputs.file_system }}
tpm_enabled: ${{ inputs.tpm_enabled }}
- name: Clean up after test
Expand Down

0 comments on commit 17725dc

Please sign in to comment.