{"action":"created","issue":{"url":"https://api.github.com/repos/docker/docker/issues/15078","labels_url":"https://api.github.com/repos/docker/docker/issues/15078/labels{/name}","comments_url":"https://api.github.com/repos/docker/docker/issues/15078/comments","events_url":"https://api.github.com/repos/docker/docker/issues/15078/events","html_url":"https://github.com/docker/docker/pull/15078","id":97659797,"number":15078,"title":"Implement docker update command","user":{"login":"hqhq","id":6415670,"avatar_url":"https://avatars.githubusercontent.com/u/6415670?v=3","gravatar_id":"","url":"https://api.github.com/users/hqhq","html_url":"https://github.com/hqhq","followers_url":"https://api.github.com/users/hqhq/followers","following_url":"https://api.github.com/users/hqhq/following{/other_user}","gists_url":"https://api.github.com/users/hqhq/gists{/gist_id}","starred_url":"https://api.github.com/users/hqhq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hqhq/subscriptions","organizations_url":"https://api.github.com/users/hqhq/orgs","repos_url":"https://api.github.com/users/hqhq/repos","events_url":"https://api.github.com/users/hqhq/events{/privacy}","received_events_url":"https://api.github.com/users/hqhq/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/docker/docker/labels/impact/api","name":"impact/api","color":"bfe5bf"},{"url":"https://api.github.com/repos/docker/docker/labels/impact/changelog","name":"impact/changelog","color":"bfe5bf"},{"url":"https://api.github.com/repos/docker/docker/labels/impact/cli","name":"impact/cli","color":"bfe5bf"},{"url":"https://api.github.com/repos/docker/docker/labels/status/4-merge","name":"status/4-merge","color":"d4c5f9"}],"state":"closed","locked":false,"assignee":{"login":"duglin","id":1944671,"avatar_url":"https://avatars.githubusercontent.com/u/1944671?v=3","gravatar_id":"","url":"https://api.github.com/users/duglin","html_url":"https://github.com/duglin","followers_url":"https://api.github.com/users/duglin/followers","following_url":"https://api.github.com/users/duglin/following{/other_user}","gists_url":"https://api.github.com/users/duglin/gists{/gist_id}","starred_url":"https://api.github.com/users/duglin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/duglin/subscriptions","organizations_url":"https://api.github.com/users/duglin/orgs","repos_url":"https://api.github.com/users/duglin/repos","events_url":"https://api.github.com/users/duglin/events{/privacy}","received_events_url":"https://api.github.com/users/duglin/received_events","type":"User","site_admin":false},"milestone":null,"comments":88,"created_at":"2015-07-28T09:46:31Z","updated_at":"2015-12-28T17:13:00Z","closed_at":"2015-12-28T16:55:32Z","pull_request":{"url":"https://api.github.com/repos/docker/docker/pulls/15078","html_url":"https://github.com/docker/docker/pull/15078","diff_url":"https://github.com/docker/docker/pull/15078.diff","patch_url":"https://github.com/docker/docker/pull/15078.patch"},"body":"fixes #6323\r\n\r\nIt's used for change properties of one or more containers, we only\r\nsupport resource configs for now. It can be extended in the future.\r\n\r\nSigned-off-by: Qiang Huang <h.huangqiang@huawei.com>\r\n\r\n-------------------------\r\n\r\nThe original PR #11442, compared to the former one, changelog:\r\n - support set for both running and stopped containers\r\n - support more cgroup configs as the new ones were merged\r\n - move some container config change logic to container daemon/container_xxx.go\r\n - addressed all issues pointed by @duglin \r\n\r\nWe have strong needs for this feature for a long time, see #3285. It's really useful feature and necessary for most product usage.\r\n\r\n@icecrime @crosbymichael as we discussed on DockerCon, we should reconsider this feature, I have pinged you guys several times hope we can reopen the original one but got no reply, so I opened this new PR, hope it's ok. If the old one can be reopened, I'm ok to close this and continue the original discussion."},"comment":{"url":"https://api.github.com/repos/docker/docker/issues/comments/167605617","html_url":"https://github.com/docker/docker/pull/15078#issuecomment-167605617","issue_url":"https://api.github.com/repos/docker/docker/issues/15078","id":167605617,"user":{"login":"thaJeztah","id":1804568,"avatar_url":"https://avatars.githubusercontent.com/u/1804568?v=3","gravatar_id":"","url":"https://api.github.com/users/thaJeztah","html_url":"https://github.com/thaJeztah","followers_url":"https://api.github.com/users/thaJeztah/followers","following_url":"https://api.github.com/users/thaJeztah/following{/other_user}","gists_url":"https://api.github.com/users/thaJeztah/gists{/gist_id}","starred_url":"https://api.github.com/users/thaJeztah/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thaJeztah/subscriptions","organizations_url":"https://api.github.com/users/thaJeztah/orgs","repos_url":"https://api.github.com/users/thaJeztah/repos","events_url":"https://api.github.com/users/thaJeztah/events{/privacy}","received_events_url":"https://api.github.com/users/thaJeztah/received_events","type":"User","site_admin":false},"created_at":"2015-12-28T17:12:59Z","updated_at":"2015-12-28T17:12:59Z","body":"@moxiegirl I noticed some tiny nits in the docs, but will create a follow up PR to address those. I didn't think it was worth blocking the merge on those :)"}}