biertie opened a new pull request #7452: [AIRFLOW-6831] Support for various ssh 
keys and certificates
URL: https://github.com/apache/airflow/pull/7452
 
 
   
   For our usecase need to support both ed25519 keys and 'MFA', using a signed 
key.
   This pull request adds the possibility to both add support for these 2 
things.
   
   When creating a hook some extra parameters become availble in the 
'extra_parameters' hash, notably:
   
       key_type: (str) can be rsa, ecdsa or ed25519; defaults to rsa
       cert_file: (str) path towards the certicate. Defaults to None
   
   The only thing I am not sure about yet is the part where we read the config 
file from ~/.ssh. 
   There we default to rsa. Is it ok? Or should we also try to figure out what 
keys are used? 
   
   This also rewrites the whole concept of the sshHOOK, and how it uses the 
keys. At the moment it is only tested in our own infrastructure. 
   
   Bert

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to