airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (Jira)" <>
Subject [jira] [Commented] (AIRFLOW-4574) Add SSHHook private key parameter pkey
Date Fri, 18 Oct 2019 13:08:00 GMT


ASF subversion and git services commented on AIRFLOW-4574:

Commit 7e9075f365bf9c8f287db5a6304b83dfaa5458f0 in airflow's branch refs/heads/v1-10-stable
from dstandish
[;h=7e9075f ]

[AIRFLOW-4574] SSHHook private_key may only be supplied in extras (#6163)

* discussion on original PR suggested removing private_key option as init param
* with this PR, can still provide through extras, but not as init param
* also add support for private_key in tunnel -- missing in original PR for this issue
* remove test related to private_key init param
* use context manager to auto-close socket listener so tests can be re-run

(cherry picked from commit 0790ede70270ceb8b92f8c8df530361749221053)

> Add SSHHook private key parameter pkey
> --------------------------------------
>                 Key: AIRFLOW-4574
>                 URL:
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: hooks
>            Reporter: Freddy Fostvedt
>            Assignee: Freddy Fostvedt
>            Priority: Minor
>             Fix For: 1.10.6
> The SSHHook only supports key_file parameter for specifying the path to a private key
on disk. This means that private keys for connections that use ssh hooks must be stored on
the disk of the worker instead of in the connection database. Maintaining the relationship
between the worker's disk state and the connection makes deploying connection changes unnecessarily
> Paramiko, which SSHHook is built on, has support for accepting private keys as an input
parameter (pkey)
> []
> The work involved in doing this should only be to add pkey as a parameter to the SSHHook
constructor, and test that SSHConnection passes pkey to SSHHook which then passes it to Paramiko.

This message was sent by Atlassian Jira

View raw message