ci(core): increase device_tests timeout to 50m on T3W1
What changed, and why it matters
This commit only adjusts how long automated test jobs are allowed to run in GitHub Actions for one Trezor hardware model (T3W1). It extends a pytest session timeout from 40 minutes to 50 minutes and the overall CI job timeout from 50 minutes to 60 minutes, but only for the T3W1 model. There is no change to firmware code, cryptography, user data handling, or device behavior. It is purely a CI configuration tweak to prevent false timeout failures when running tests with two workers.
No security action needed. This is a benign CI timeout adjustment.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies .github/workflows/core.yml. It changes TESTOPTS session-timeout from 2400s to 3000s when matrix.model == ‘T3W1’, and changes the GitHub Actions timeout-minutes from a fixed 50 to a conditional expression: 60 for T3W1, otherwise 40. The change is explained by the commit message as fixing UI test timeouts when running with 2 workers on T3W1. No firmware, bootloader, crypto, or application code is touched.
Changed components
.github/workflows/core.ymlInspect captured patch +2 / −2
diff --git a/.github/workflows/core.yml b/.github/workflows/core.yml
index af0687ab..6af34378 100644
--- a/.github/workflows/core.yml
+++ b/.github/workflows/core.yml
@@ -320,8 +320,8 @@ jobs:
PYTEST_TIMEOUT: ${{ matrix.asan == 'asan' && 600 || 400 }}
ACTIONS_DO_UI_TEST: ${{ matrix.coins == 'universal' && matrix.asan == 'noasan' }}
TEST_LANG: ${{ matrix.lang }}
- TESTOPTS: "--durations 10 --session-timeout ${{ matrix.model == 'T3W1' && '2400' || '1800' }}" # pytest global timeout
- timeout-minutes: 50 # CI job timeout
+ TESTOPTS: "--durations 10 --session-timeout ${{ matrix.model == 'T3W1' && '3000' || '1800' }}" # pytest global timeout
+ timeout-minutes: ${{ matrix.model == 'T3W1' && 60 || 40 }} # CI job timeout
steps:
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # actions/checkout@v6.0.2
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.