potiuk opened a new pull request, #67829:
URL: https://github.com/apache/airflow/pull/67829
`test_command_timeout_fail` set `cmd_timeout=0.001`, which the SSH hook
forwards into paramiko's `exec_command`. paramiko applies that value to
**opening the channel** as well as reading command output, so on a loaded CI
runner opening the channel exceeds 1 ms and raises `SSHException("Timeout
opening channel.")` *before* the hook's command-timeout logic runs — instead of
the expected `AirflowException`. This flaked in the scheduled [Tests (AMD) run
26732900291](https://github.com/apache/airflow/actions/runs/26732900291/attempts/1)
(Low dep tests: providers) and passed on rerun.
Fix: use a `cmd_timeout` large enough to reliably open the channel (`0.5s`)
and a longer command (`sleep 5`) so the read loop is what times out, keeping
the test on its intended `AirflowException` path.
No newsfragment: providers do not consume them, and this is a test-only
change.
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Opus 4.8)
Generated-by: Claude Code (Opus 4.8) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
