ci(core): hw: power cycle all ports on PPPS hub
What changed, and why it matters
This commit changes a CI (Continuous Integration) workflow file for hardware testing. It modifies the configuration so that a USB hub power-cycles all ports instead of just port 1 during automated test runs. There is no change to the actual Trezor firmware, wallet application, or any code that end users run. This is purely a test infrastructure tweak.
No security action required. This is a CI maintenance change. Reviewers may verify that power-cycling all ports does not disrupt shared CI hardware, but no product security review is indicated.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff updates .github/workflows/core-hw.yml, changing the TT_UHUB_PORT environment variable from 1 to all in two hardware test jobs. This tells the PPPS (Per-Port Power Switching) USB hub used in the CI hardware test farm to cycle power on all ports rather than only port 1. The change affects no source code, build artifacts, cryptographic operations, device communication protocol, or user-facing behavior.
Changed components
.github/workflows/core-hw.ymlCI hardware test infrastructure configurationInspect captured patch +2 / −2
diff --git a/.github/workflows/core-hw.yml b/.github/workflows/core-hw.yml
index 4485d854..ecbe8977 100644
--- a/.github/workflows/core-hw.yml
+++ b/.github/workflows/core-hw.yml
@@ -45,7 +45,7 @@ jobs:
-k 'not test_authenticate_device'
--durations=50
--session-timeout 19800
- TT_UHUB_PORT: 1
+ TT_UHUB_PORT: all
timeout-minutes: 360 # 6h CI job timeout
steps:
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # actions/checkout@v4.3.1
@@ -87,7 +87,7 @@ jobs:
PYOPT: 0
DISABLE_OPTIGA: 1
BOOTLOADER_DEVEL: ${{ matrix.model == 'T2T1' && '0' || '1' }}
- TT_UHUB_PORT: 1
+ TT_UHUB_PORT: all
steps:
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # actions/checkout@v4.3.1
with:
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.