nshetty15 commented on a change in pull request #19981:
URL: https://github.com/apache/airflow/pull/19981#discussion_r778848170
##########
File path: airflow/providers/ssh/hooks/ssh.py
##########
@@ -257,58 +259,74 @@ def get_conn(self) -> paramiko.SSHClient:
:rtype: paramiko.client.SSHClient
"""
self.log.debug('Creating SSH client for conn_id: %s', self.ssh_conn_id)
- client = paramiko.SSHClient()
- if not self.allow_host_key_change:
- self.log.warning(
- "Remote Identification Change is not verified. "
- "This won't protect against Man-In-The-Middle attacks"
- )
- client.load_system_host_keys()
+ max_time_to_wait = 10
+ for time_to_wait in self._exponential_sleep_generator(initial=1,
maximum=max_time_to_wait):
Review comment:
okay.. let me change this to use tenacity.
--
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]
