Skip to content

Commit

Permalink
GitHub Actions: use report_name instead of variables in publish-logs (l…
Browse files Browse the repository at this point in the history
…f-edge#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 <uncle.decart@gmail.com>
  • 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.