{"action":"created","issue":{"url":"https://api.github.com/repos/cloudfoundry/warden/issues/101","labels_url":"https://api.github.com/repos/cloudfoundry/warden/issues/101/labels{/name}","comments_url":"https://api.github.com/repos/cloudfoundry/warden/issues/101/comments","events_url":"https://api.github.com/repos/cloudfoundry/warden/issues/101/events","html_url":"https://github.com/cloudfoundry/warden/issues/101","id":101957735,"number":101,"title":"Remove Idle timeouts","user":{"login":"DanLavine","id":5760002,"avatar_url":"https://avatars.githubusercontent.com/u/5760002?v=3","gravatar_id":"","url":"https://api.github.com/users/DanLavine","html_url":"https://github.com/DanLavine","followers_url":"https://api.github.com/users/DanLavine/followers","following_url":"https://api.github.com/users/DanLavine/following{/other_user}","gists_url":"https://api.github.com/users/DanLavine/gists{/gist_id}","starred_url":"https://api.github.com/users/DanLavine/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DanLavine/subscriptions","organizations_url":"https://api.github.com/users/DanLavine/orgs","repos_url":"https://api.github.com/users/DanLavine/repos","events_url":"https://api.github.com/users/DanLavine/events{/privacy}","received_events_url":"https://api.github.com/users/DanLavine/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-08-19T17:49:10Z","updated_at":"2015-09-01T00:29:58Z","closed_at":"2015-09-01T00:29:58Z","body":"The concerns for this issue are:\r\n\r\n* Idle timeout is not safe for clients.\r\n* The timeout occurs while a client is attepting to call.\r\n* It is also not thread safe.\r\n\r\nThe original PR can be found [here](https://github.com/cloudfoundry/warden/pull/93).\r\n\r\nThe PR has been reverted a second time at Warden [commit](https://github.com/cloudfoundry/warden/commit/9c9f70be01fa109a40f4c7fd63f8792dc9fedda8) and the corresponding DEA [commit]( https://github.com/cloudfoundry/dea_ng/commit/db3de2ad2be939475a0c80d3c940efef1a44dd9a).\r\n\r\nThe reason we reverted the commits is due to the fact that DEA's started to crash after a couple of hours under load. The root problem seems to be that connections between the DEA and Warden are not being properly closed and the DEA relied on the timeout connections to clean things up. We believe a better solution is to go through and make sure that all of the connections that are created between these components are correctly receiving a close connection call.\r\n\r\nNote: Some offending log lines that appeared had the message `Too many open files`."},"comment":{"url":"https://api.github.com/repos/cloudfoundry/warden/issues/comments/136536719","html_url":"https://github.com/cloudfoundry/warden/issues/101#issuecomment-136536719","issue_url":"https://api.github.com/repos/cloudfoundry/warden/issues/101","id":136536719,"user":{"login":"jpalermo","id":16798,"avatar_url":"https://avatars.githubusercontent.com/u/16798?v=3","gravatar_id":"","url":"https://api.github.com/users/jpalermo","html_url":"https://github.com/jpalermo","followers_url":"https://api.github.com/users/jpalermo/followers","following_url":"https://api.github.com/users/jpalermo/following{/other_user}","gists_url":"https://api.github.com/users/jpalermo/gists{/gist_id}","starred_url":"https://api.github.com/users/jpalermo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jpalermo/subscriptions","organizations_url":"https://api.github.com/users/jpalermo/orgs","repos_url":"https://api.github.com/users/jpalermo/repos","events_url":"https://api.github.com/users/jpalermo/events{/privacy}","received_events_url":"https://api.github.com/users/jpalermo/received_events","type":"User","site_admin":false},"created_at":"2015-09-01T00:29:58Z","updated_at":"2015-09-01T00:29:58Z","body":"Merged along with https://github.com/cloudfoundry/warden/issues/101"}}