{"action":"created","issue":{"url":"https://api.github.com/repos/jenkinsci/docker-plugin/issues/258","labels_url":"https://api.github.com/repos/jenkinsci/docker-plugin/issues/258/labels{/name}","comments_url":"https://api.github.com/repos/jenkinsci/docker-plugin/issues/258/comments","events_url":"https://api.github.com/repos/jenkinsci/docker-plugin/issues/258/events","html_url":"https://github.com/jenkinsci/docker-plugin/issues/258","id":92588844,"number":258,"title":"\"idle timeout\" can be set to 0 in DockerOnceRetentionStrategy","user":{"login":"abustany","id":2526296,"avatar_url":"https://avatars.githubusercontent.com/u/2526296?v=3","gravatar_id":"","url":"https://api.github.com/users/abustany","html_url":"https://github.com/abustany","followers_url":"https://api.github.com/users/abustany/followers","following_url":"https://api.github.com/users/abustany/following{/other_user}","gists_url":"https://api.github.com/users/abustany/gists{/gist_id}","starred_url":"https://api.github.com/users/abustany/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abustany/subscriptions","organizations_url":"https://api.github.com/users/abustany/orgs","repos_url":"https://api.github.com/users/abustany/repos","events_url":"https://api.github.com/users/abustany/events{/privacy}","received_events_url":"https://api.github.com/users/abustany/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-02T09:08:15Z","updated_at":"2015-07-02T13:15:50Z","closed_at":null,"body":"We've been running the Docker plugin since quite early versions (0.5 I'd say), and are now running it 0.8 on Jenkins 1.617. It looks like the \"idle timeout\" field of the DockerOnceRetentionStrategy got added at some point, and if not defined gets a default value of 0. That in turns means that if you're unlucky enough to get the CloudRetentionStrategy.check run while your container is still not fully up, the container will get killed.\r\n\r\nThe plugin should probably set a reasonable default value for \"idle timeout\" in case it's set to 0.\r\n\r\nAlso, not completely related, but is there a reason for using `PortUtils.canConnect(host, port).withEveryRetryWaitFor` instead of `PortUtils.canConnect(host, port).bySshWithEveryRetryWaitFor` in `DockerComputerLauncher` ? I'd say it'd make more sense to wait until SSH is really up, rather than just the container being up and Docker accepting connections without sshd being ready yet in the container..."},"comment":{"url":"https://api.github.com/repos/jenkinsci/docker-plugin/issues/comments/118028799","html_url":"https://github.com/jenkinsci/docker-plugin/issues/258#issuecomment-118028799","issue_url":"https://api.github.com/repos/jenkinsci/docker-plugin/issues/258","id":118028799,"user":{"login":"KostyaSha","id":231611,"avatar_url":"https://avatars.githubusercontent.com/u/231611?v=3","gravatar_id":"","url":"https://api.github.com/users/KostyaSha","html_url":"https://github.com/KostyaSha","followers_url":"https://api.github.com/users/KostyaSha/followers","following_url":"https://api.github.com/users/KostyaSha/following{/other_user}","gists_url":"https://api.github.com/users/KostyaSha/gists{/gist_id}","starred_url":"https://api.github.com/users/KostyaSha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KostyaSha/subscriptions","organizations_url":"https://api.github.com/users/KostyaSha/orgs","repos_url":"https://api.github.com/users/KostyaSha/repos","events_url":"https://api.github.com/users/KostyaSha/events{/privacy}","received_events_url":"https://api.github.com/users/KostyaSha/received_events","type":"User","site_admin":false},"created_at":"2015-07-02T13:15:50Z","updated_at":"2015-07-02T13:15:50Z","body":"DockerOnceRetentionStrategy is designed to have idle = 0. This strategy appeared here only because of \"workflow\" \"support\". It probably makes to try other strategy. If error persist, then issue not in docker implementation. I'm closing this, because such things are known and discussed many times on other issues."}}