[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-09-08 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r704902758 ## File path: tests/providers/ssh/operators/test_ssh.py ## @@ -138,7 +160,9 @@ def test_no_output_command(self): def test_arg_checking(self):

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-09-08 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r704902758 ## File path: tests/providers/ssh/operators/test_ssh.py ## @@ -138,7 +160,9 @@ def test_no_output_command(self): def test_arg_checking(self):

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-09-07 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r703713799 ## File path: tests/providers/ssh/operators/test_ssh.py ## @@ -138,7 +160,9 @@ def test_no_output_command(self): def test_arg_checking(self):

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-08-24 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r694784041 ## File path: tests/providers/ssh/hooks/test_ssh.py ## @@ -475,6 +504,122 @@ def test_ssh_connection_with_no_host_key_where_no_host_key_check_is_f

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-08-08 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r684854398 ## File path: tests/providers/ssh/hooks/test_ssh.py ## @@ -475,6 +504,122 @@ def test_ssh_connection_with_no_host_key_where_no_host_key_check_is_f

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-08-05 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r683804274 ## File path: docs/apache-airflow-providers-ssh/connections/ssh.rst ## @@ -47,7 +47,8 @@ Extra (optional) * ``key_file`` - Full Path of the pr

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-08-05 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r683763029 ## File path: airflow/providers/ssh/operators/ssh.py ## @@ -77,9 +87,24 @@ def __init__( self.remote_host = remote_host self.comm

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-08-01 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r680573896 ## File path: tests/providers/ssh/hooks/test_ssh.py ## @@ -475,6 +504,120 @@ def test_ssh_connection_with_no_host_key_where_no_host_key_check_is_f

[GitHub] [airflow] fatmumuhomer commented on a change in pull request #17236: [Airflow 16364] Add conn_timeout and cmd_timeout params to SSHOperator; add conn_timeout param to SSHHook

2021-07-28 Thread GitBox
fatmumuhomer commented on a change in pull request #17236: URL: https://github.com/apache/airflow/pull/17236#discussion_r678255423 ## File path: airflow/providers/ssh/hooks/ssh.py ## @@ -101,7 +103,8 @@ def __init__( password: Optional[str] = None, key_file: O