{"action":"created","issue":{"url":"https://api.github.com/repos/cylc/cylc/issues/1645","labels_url":"https://api.github.com/repos/cylc/cylc/issues/1645/labels{/name}","comments_url":"https://api.github.com/repos/cylc/cylc/issues/1645/comments","events_url":"https://api.github.com/repos/cylc/cylc/issues/1645/events","html_url":"https://github.com/cylc/cylc/pull/1645","id":111411291,"number":1645,"title":"cylc.gui.tailer: fix SSH zombie tail","user":{"login":"matthewrmshin","id":908613,"avatar_url":"https://avatars.githubusercontent.com/u/908613?v=3","gravatar_id":"","url":"https://api.github.com/users/matthewrmshin","html_url":"https://github.com/matthewrmshin","followers_url":"https://api.github.com/users/matthewrmshin/followers","following_url":"https://api.github.com/users/matthewrmshin/following{/other_user}","gists_url":"https://api.github.com/users/matthewrmshin/gists{/gist_id}","starred_url":"https://api.github.com/users/matthewrmshin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthewrmshin/subscriptions","organizations_url":"https://api.github.com/users/matthewrmshin/orgs","repos_url":"https://api.github.com/users/matthewrmshin/repos","events_url":"https://api.github.com/users/matthewrmshin/events{/privacy}","received_events_url":"https://api.github.com/users/matthewrmshin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/cylc/cylc/milestones/42","html_url":"https://github.com/cylc/cylc/milestones/next%20release","labels_url":"https://api.github.com/repos/cylc/cylc/milestones/42/labels","id":1340072,"number":42,"title":"next release","description":"","creator":{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false},"open_issues":2,"closed_issues":14,"state":"open","created_at":"2015-10-06T11:03:31Z","updated_at":"2015-10-14T14:32:05Z","due_on":"2015-10-27T11:00:00Z","closed_at":null},"comments":0,"created_at":"2015-10-14T14:32:05Z","updated_at":"2015-10-14T14:53:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cylc/cylc/pulls/1645","html_url":"https://github.com/cylc/cylc/pull/1645","diff_url":"https://github.com/cylc/cylc/pull/1645.diff","patch_url":"https://github.com/cylc/cylc/pull/1645.patch"},"body":"On signal to `ssh` client, a signal is sent to `sshd` on server.\r\nHowever, `sshd` cannot send a signal to the `tail` command, because it\r\nis not a terminal. This leaves the `tail` command in a zombie state.\r\nApparently, we can use `ssh -t` or `ssh -tt`, but that just causes the\r\ncommand to hang for some reason, (which hangs the GUI as well). A quick\r\nhack here is to do something like `tail --pid=$PPID`, so `tail` dies as\r\nsoon as `PPID` dies.\r\n\r\n@hjoliver @arjclark please review."},"comment":{"url":"https://api.github.com/repos/cylc/cylc/issues/comments/148074769","html_url":"https://github.com/cylc/cylc/pull/1645#issuecomment-148074769","issue_url":"https://api.github.com/repos/cylc/cylc/issues/1645","id":148074769,"user":{"login":"arjclark","id":2562650,"avatar_url":"https://avatars.githubusercontent.com/u/2562650?v=3","gravatar_id":"","url":"https://api.github.com/users/arjclark","html_url":"https://github.com/arjclark","followers_url":"https://api.github.com/users/arjclark/followers","following_url":"https://api.github.com/users/arjclark/following{/other_user}","gists_url":"https://api.github.com/users/arjclark/gists{/gist_id}","starred_url":"https://api.github.com/users/arjclark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arjclark/subscriptions","organizations_url":"https://api.github.com/users/arjclark/orgs","repos_url":"https://api.github.com/users/arjclark/repos","events_url":"https://api.github.com/users/arjclark/events{/privacy}","received_events_url":"https://api.github.com/users/arjclark/received_events","type":"User","site_admin":false},"created_at":"2015-10-14T14:53:50Z","updated_at":"2015-10-14T14:53:50Z","body":"Review 2 - looks good to me. Seems to have fixed the problem I reported."}}