fix(ci/hw-test): downgrade checkout and upload-artifact actions
What changed, and why it matters
This commit simply downgrades two GitHub Actions helper tools used in automated hardware testing workflows. The change is described as fixing test failures caused by incompatible runners. There is no indication this affects the Trezor firmware itself, user funds, or device security.
No security action required. Treat as routine CI maintenance. If security concerns arise, verify the pinned action commit hashes against the official actions/checkout and actions/upload-artifact repositories.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit pins older versions of actions/checkout (v6.0.2 → v4.3.1) and actions/upload-artifact (v7.0.0 → v4.6.2) in .github/workflows/core-hw.yml. The stated reason is that hardware tests began failing after a recent actions upgrade, likely due to runner incompatibility. This is a CI maintenance change; it does not modify firmware, cryptographic, or wallet code.
Changed components
.github/workflows/core-hw.ymlInspect captured patch +6 / −6
diff --git a/.github/workflows/core-hw.yml b/.github/workflows/core-hw.yml
index 73ea9ddb..ca657352 100644
--- a/.github/workflows/core-hw.yml
+++ b/.github/workflows/core-hw.yml
@@ -52,7 +52,7 @@ jobs:
QUIET_MODE: 1
timeout-minutes: 360 # 6h CI job timeout
steps:
- - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # actions/checkout@v6.0.2
+ - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # actions/checkout@v4.3.1
with:
submodules: recursive
- uses: ./.github/actions/environment
@@ -66,7 +66,7 @@ jobs:
nix-shell --run "uv run pytest -v --verbose-log-file pytest.log tests/device_tests $TESTOPTS"
- run: tail -n50 trezor.log || true
if: failure()
- - uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # actions/upload-artifact@v7.0.0
+ - uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # actions/upload-artifact@v4.6.2
with:
name: core-hardware-${{ matrix.model }}-${{ matrix.coins }}
path: |
@@ -95,7 +95,7 @@ jobs:
TT_UHUB_PORT: 1
QUIET_MODE: 1
steps:
- - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # actions/checkout@v6.0.2
+ - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # actions/checkout@v4.3.1
with:
submodules: recursive
- uses: ./.github/actions/environment
@@ -109,7 +109,7 @@ jobs:
nix-shell --arg fullDeps true --run "./core/tests/run_tests_device_emu_monero.sh --trezor-path webusb:"
- run: tail -n50 trezor.log || true
if: failure()
- - uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # actions/upload-artifact@v7.0.0
+ - uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # actions/upload-artifact@v4.6.2
with:
name: core-hardware-monero-${{ matrix.model }}
path: trezor.log
@@ -134,7 +134,7 @@ jobs:
BITCOIN_ONLY: ${{ matrix.coins == 'universal' && '0' || '1' }}
DEBUG_LINK: 1
steps:
- - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # actions/checkout@v6.0.2
+ - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # actions/checkout@v4.3.1
with:
submodules: recursive
- uses: ./.github/actions/environment
@@ -142,7 +142,7 @@ jobs:
- run: nix-shell --run "uv run legacy/script/setup"
- run: nix-shell --run "export PRODUCTION=0 && uv run legacy/script/cibuild"
- run: nix-shell --arg hardwareTest true --run "ci/hardware_tests/t1_hw_test.sh"
- - uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # actions/upload-artifact@v7.0.0
+ - uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # actions/upload-artifact@v4.6.2
with:
name: legacy-hardware-${{ matrix.coins }}
path: ci/hardware_tests/*.mp4
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.