[GitHub] [airflow] Aakcht commented on pull request #29347: SSH Provider: Add cmd_timeout to ssh connection extra

2023-02-22 Thread via GitHub
Aakcht commented on PR #29347: URL: https://github.com/apache/airflow/pull/29347#issuecomment-1439863842 Hi, @pollynesterovich . This PR is not related to the chart since it added a fix to airflow provider. So as I understand, for it to get to the chart, firstly a new 3.5.0 [SSH provider v

[GitHub] [airflow] Aakcht commented on pull request #29347: SSH Provider: Add cmd_timeout to ssh connection extra

2023-02-19 Thread via GitHub
Aakcht commented on PR #29347: URL: https://github.com/apache/airflow/pull/29347#issuecomment-1436048563 Looks like the unsuccessful check is not related to this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [airflow] Aakcht commented on pull request #29347: SSH Provider: Add cmd_timeout to ssh connection extra

2023-02-17 Thread via GitHub
Aakcht commented on PR #29347: URL: https://github.com/apache/airflow/pull/29347#issuecomment-1434481893 Hi, @ferruzzi do you have any questions left? I'd be happy to help. @eladkal @potiuk I will be glad could take a look at this PR too, cause I see you reviewed #27184, which is rel

[GitHub] [airflow] Aakcht commented on pull request #29347: SSH Provider: Add cmd_timeout to ssh connection extra

2023-02-15 Thread via GitHub
Aakcht commented on PR #29347: URL: https://github.com/apache/airflow/pull/29347#issuecomment-1431948430 @ferruzzi I'd say that this PR is about the timeout for waiting for command reply. For example, consider the command ```bash sleep 10 ``` (as actually used in the tests for