{"action":"created","issue":{"url":"https://api.github.com/repos/docker/docker/issues/14452","labels_url":"https://api.github.com/repos/docker/docker/issues/14452/labels{/name}","comments_url":"https://api.github.com/repos/docker/docker/issues/14452/comments","events_url":"https://api.github.com/repos/docker/docker/issues/14452/events","html_url":"https://github.com/docker/docker/pull/14452","id":93679749,"number":14452,"title":"Cleanup exited detached exec jobs","user":{"login":"icecrime","id":1564054,"avatar_url":"https://avatars.githubusercontent.com/u/1564054?v=3","gravatar_id":"","url":"https://api.github.com/users/icecrime","html_url":"https://github.com/icecrime","followers_url":"https://api.github.com/users/icecrime/followers","following_url":"https://api.github.com/users/icecrime/following{/other_user}","gists_url":"https://api.github.com/users/icecrime/gists{/gist_id}","starred_url":"https://api.github.com/users/icecrime/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icecrime/subscriptions","organizations_url":"https://api.github.com/users/icecrime/orgs","repos_url":"https://api.github.com/users/icecrime/repos","events_url":"https://api.github.com/users/icecrime/events{/privacy}","received_events_url":"https://api.github.com/users/icecrime/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/docker/docker/labels/status/1-needs-design-review","name":"status/1-needs-design-review","color":"fef2c0"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-07-08T03:08:49Z","updated_at":"2015-07-08T15:47:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/docker/docker/pulls/14452","html_url":"https://github.com/docker/docker/pull/14452","diff_url":"https://github.com/docker/docker/pull/14452.diff","patch_url":"https://github.com/docker/docker/pull/14452.patch"},"body":"Exec is a second class citizen that should mostly be used for debugging\r\npurpose. It doesn't offer the same attach/wait/delete functionalities\r\nthat containers do which makes it impossible to implement the proper\r\nclient workflow of retrieving the exitCode before properly deleting all\r\ntraces of the exec structure.\r\n\r\nThis patch aims at offering a workaround _without_ expanding the scope\r\nof exec functionalities by making the execConfig cleanup automatic for\r\nany detached exec job. This is basically saying that you cannot retrieve\r\nthe exitStatus of a detached exec job, which I think is an acceptable\r\ntradeoff.\r\n\r\nThis could be a partial resolution for #14444."},"comment":{"url":"https://api.github.com/repos/docker/docker/issues/comments/119631252","html_url":"https://github.com/docker/docker/pull/14452#issuecomment-119631252","issue_url":"https://api.github.com/repos/docker/docker/issues/14452","id":119631252,"user":{"login":"thockin","id":5595220,"avatar_url":"https://avatars.githubusercontent.com/u/5595220?v=3","gravatar_id":"","url":"https://api.github.com/users/thockin","html_url":"https://github.com/thockin","followers_url":"https://api.github.com/users/thockin/followers","following_url":"https://api.github.com/users/thockin/following{/other_user}","gists_url":"https://api.github.com/users/thockin/gists{/gist_id}","starred_url":"https://api.github.com/users/thockin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thockin/subscriptions","organizations_url":"https://api.github.com/users/thockin/orgs","repos_url":"https://api.github.com/users/thockin/repos","events_url":"https://api.github.com/users/thockin/events{/privacy}","received_events_url":"https://api.github.com/users/thockin/received_events","type":"User","site_admin":false},"created_at":"2015-07-08T15:47:03Z","updated_at":"2015-07-08T15:47:03Z","body":"@cpuguy83 yeah, I was ignoring the details of the PR in favor of the description.  The vast majority of what we do has no stdin, but we absolutely care about stdout and stderr."}}