Skip to content

Commit

Permalink
ci: actions: don't fill logs with junk when uploading to s3
Browse files Browse the repository at this point in the history
  • Loading branch information
mmilata committed May 9, 2024
1 parent 067daa7 commit 946e855
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
4 changes: 2 additions & 2 deletions .github/actions/ui-report/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,12 +38,12 @@ runs:
shell: sh
- name: Upload report
run: |
aws s3 sync ${{ github.run_id }} s3://data.trezor.io/dev/firmware/ui_report/${{ github.run_id }}
aws s3 sync --no-progress ${{ github.run_id }} s3://data.trezor.io/dev/firmware/ui_report/${{ github.run_id }}
echo "[UI test report](https://data.trezor.io/dev/firmware/ui_report/${{ github.run_id }}/${{ inputs.model }}-${{ inputs.lang }}-${{ github.job }}/index.html)" >> $GITHUB_STEP_SUMMARY
shell: sh
- name: Upload test screen recording
run: |
aws s3 sync ci/ui_test_records s3://data.trezor.io/dev/firmware/ui_tests
aws s3 sync --no-progress ci/ui_test_records s3://data.trezor.io/dev/firmware/ui_tests
# TODO: generate directory listing / autoindex
shell: sh
- uses: actions/upload-artifact@v4
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/core.yml
Original file line number Diff line number Diff line change
Expand Up @@ -690,7 +690,7 @@ jobs:
fi
- name: Upload diff from main branch
run: |
aws s3 sync master_diff s3://data.trezor.io/dev/firmware/master_diff/${{ github.run_id }}
aws s3 sync --no-progress master_diff s3://data.trezor.io/dev/firmware/master_diff/${{ github.run_id }}
continue-on-error: true

core_ui_comment:
Expand All @@ -706,6 +706,7 @@ jobs:
git diff --quiet origin/main...HEAD -- tests/ui_tests/fixtures.json || echo "FIXTURES_CHANGED=$?" >> $GITHUB_OUTPUT
id: check-fixtures-changed
- uses: ./.github/actions/ui-comment
# TODO: always run if comment already exists
if: ${{ steps.check-fixtures-changed.outputs.FIXTURES_CHANGED == '1' }}

core_upload_emu:
Expand All @@ -726,7 +727,7 @@ jobs:
continue-on-error: true
- run: |
rm unix/trezor-emu-core
aws s3 sync unix s3://data.trezor.io/dev/firmware/emu-nightly
aws s3 sync --no-progress unix s3://data.trezor.io/dev/firmware/emu-nightly
# Connect
# TODO: core_connect_test
5 changes: 3 additions & 2 deletions .github/workflows/legacy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ jobs:
fi
- name: Upload main branch diff
run: |
aws s3 sync master_diff s3://data.trezor.io/dev/firmware/master_diff/${{ github.run_id }}
aws s3 sync --no-progress master_diff s3://data.trezor.io/dev/firmware/master_diff/${{ github.run_id }}
continue-on-error: true

legacy_ui_comment:
Expand All @@ -199,6 +199,7 @@ jobs:
git diff --quiet origin/main...HEAD -- tests/ui_tests/fixtures.json || echo "FIXTURES_CHANGED=$?" >> $GITHUB_OUTPUT
id: check-fixtures-changed
- uses: ./.github/actions/ui-comment
# TODO: always run if comment already exists
if: ${{ steps.check-fixtures-changed.outputs.FIXTURES_CHANGED == '1' }}

core_upload_emu:
Expand All @@ -220,4 +221,4 @@ jobs:
- run: |
mkdir emulators
cp trezor-emu-legacy* emulators
aws s3 sync emulators s3://data.trezor.io/dev/firmware/emu-nightly
aws s3 sync --no-progress emulators s3://data.trezor.io/dev/firmware/emu-nightly

0 comments on commit 946e855

Please sign in to comment.