You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2017/11/02 17:58:00 UTC

[jira] [Commented] (AIRFLOW-1779) Add keepalive packets to ssh hook

    [ https://issues.apache.org/jira/browse/AIRFLOW-1779?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16236267#comment-16236267 ] 

ASF subversion and git services commented on AIRFLOW-1779:
----------------------------------------------------------

Commit d2f9d189c6c4244467434de600aae95e2a76558e in incubator-airflow's branch refs/heads/v1-9-test from [~RJKeevil]
[ https://git-wip-us.apache.org/repos/asf?p=incubator-airflow.git;h=d2f9d18 ]

[AIRFLOW-1779] Add keepalive packets to ssh hook

Make use of paramiko's set_keepalive method to
send keepalive packets every
keepalive_interval seconds.  This will prevent
long running queries with no terminal
output from being termanated as idle, for example
by an intermediate NAT.

Set on by default with a 30 second interval.

Closes #2749 from RJKeevil/add-sshhook-keepalive

(cherry picked from commit 1bde7833854210676dcd6e8da8b6d9567e12031a)
Signed-off-by: Bolke de Bruin <bo...@xs4all.nl>


> Add keepalive packets to ssh hook
> ---------------------------------
>
>                 Key: AIRFLOW-1779
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-1779
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: contrib
>    Affects Versions: Airflow 1.8
>            Reporter: Rob Keevil
>            Assignee: Rob Keevil
>             Fix For: 1.9.0
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> The existing SSH hook does not make use of keepalive packets, and connections may thus be dropped for long running commands that do not have frequent terminal output.  Paramiko has a set_keepalive method available on Transport objects, and we should optionally set this in the SSHHook.
> I'm working on adding this, will link PR here when ready.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)