punx120 opened a new issue, #27182:
URL: https://github.com/apache/airflow/issues/27182
### Apache Airflow Provider(s)
ssh
### Versions of Apache Airflow Providers
_No response_
### Apache Airflow version
2.4.1
### Operating System
linux
### Deployment
Other
### Deployment details
_No response_
### What happened
Hi,
SSHOperator documentation states that we should be using cmd_timeout instead
of timeout
```
:param timeout: (deprecated) timeout (in seconds) for executing the command.
The default is 10 seconds.
Use conn_timeout and cmd_timeout parameters instead.
```
But the code doesn't use cmd_timeout at all - and it's still passing
`self.timeout` when running the ssh command:
```
return self.ssh_hook.exec_ssh_client_command(
ssh_client, command, timeout=self.timeout,
environment=self.environment, get_pty=self.get_pty
)
```
It seems to me that we should `self.cmd_timeout` here instead. When creating
the hook, it correctly uses `self.conn_timeout`.
I'll try to work on a PR for this.
### What you think should happen instead
_No response_
### How to reproduce
_No response_
### Anything else
_No response_
### Are you willing to submit PR?
- [X] Yes I am willing to submit a PR!
### Code of Conduct
- [X] I agree to follow this project's [Code of
Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
--
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]
