{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/42061","repository_url":"https://api.github.com/repos/kubernetes/kubernetes","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/42061/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/42061/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/42061/events","html_url":"https://github.com/kubernetes/kubernetes/issues/42061","id":210106334,"number":42061,"title":"On update Kubelet starts the new manifest-run pod before fully stopping the old one  ","user":{"login":"mwielgus","id":11994812,"avatar_url":"https://avatars.githubusercontent.com/u/11994812?v=3","gravatar_id":"","url":"https://api.github.com/users/mwielgus","html_url":"https://github.com/mwielgus","followers_url":"https://api.github.com/users/mwielgus/followers","following_url":"https://api.github.com/users/mwielgus/following{/other_user}","gists_url":"https://api.github.com/users/mwielgus/gists{/gist_id}","starred_url":"https://api.github.com/users/mwielgus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mwielgus/subscriptions","organizations_url":"https://api.github.com/users/mwielgus/orgs","repos_url":"https://api.github.com/users/mwielgus/repos","events_url":"https://api.github.com/users/mwielgus/events{/privacy}","received_events_url":"https://api.github.com/users/mwielgus/received_events","type":"User","site_admin":false},"labels":[{"id":116719829,"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/area/kubelet","name":"area/kubelet","color":"0052cc","default":false},{"id":114528223,"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/priority/important-soon","name":"priority/important-soon","color":"eb6420","default":false},{"id":173493665,"url":"https://api.github.com/repos/kubernetes/kubernetes/labels/sig/node","name":"sig/node","color":"d2b48c","default":false}],"state":"open","locked":false,"assignee":{"login":"vishh","id":7063592,"avatar_url":"https://avatars.githubusercontent.com/u/7063592?v=3","gravatar_id":"","url":"https://api.github.com/users/vishh","html_url":"https://github.com/vishh","followers_url":"https://api.github.com/users/vishh/followers","following_url":"https://api.github.com/users/vishh/following{/other_user}","gists_url":"https://api.github.com/users/vishh/gists{/gist_id}","starred_url":"https://api.github.com/users/vishh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vishh/subscriptions","organizations_url":"https://api.github.com/users/vishh/orgs","repos_url":"https://api.github.com/users/vishh/repos","events_url":"https://api.github.com/users/vishh/events{/privacy}","received_events_url":"https://api.github.com/users/vishh/received_events","type":"User","site_admin":false},"assignees":[{"login":"vishh","id":7063592,"avatar_url":"https://avatars.githubusercontent.com/u/7063592?v=3","gravatar_id":"","url":"https://api.github.com/users/vishh","html_url":"https://github.com/vishh","followers_url":"https://api.github.com/users/vishh/followers","following_url":"https://api.github.com/users/vishh/following{/other_user}","gists_url":"https://api.github.com/users/vishh/gists{/gist_id}","starred_url":"https://api.github.com/users/vishh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vishh/subscriptions","organizations_url":"https://api.github.com/users/vishh/orgs","repos_url":"https://api.github.com/users/vishh/repos","events_url":"https://api.github.com/users/vishh/events{/privacy}","received_events_url":"https://api.github.com/users/vishh/received_events","type":"User","site_admin":false},{"login":"dchen1107","id":7740897,"avatar_url":"https://avatars.githubusercontent.com/u/7740897?v=3","gravatar_id":"","url":"https://api.github.com/users/dchen1107","html_url":"https://github.com/dchen1107","followers_url":"https://api.github.com/users/dchen1107/followers","following_url":"https://api.github.com/users/dchen1107/following{/other_user}","gists_url":"https://api.github.com/users/dchen1107/gists{/gist_id}","starred_url":"https://api.github.com/users/dchen1107/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dchen1107/subscriptions","organizations_url":"https://api.github.com/users/dchen1107/orgs","repos_url":"https://api.github.com/users/dchen1107/repos","events_url":"https://api.github.com/users/dchen1107/events{/privacy}","received_events_url":"https://api.github.com/users/dchen1107/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/kubernetes/kubernetes/milestones/31","html_url":"https://github.com/kubernetes/kubernetes/milestone/31","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/milestones/31/labels","id":2130587,"number":31,"title":"v1.6","description":null,"creator":{"login":"saad-ali","id":10052848,"avatar_url":"https://avatars.githubusercontent.com/u/10052848?v=3","gravatar_id":"","url":"https://api.github.com/users/saad-ali","html_url":"https://github.com/saad-ali","followers_url":"https://api.github.com/users/saad-ali/followers","following_url":"https://api.github.com/users/saad-ali/following{/other_user}","gists_url":"https://api.github.com/users/saad-ali/gists{/gist_id}","starred_url":"https://api.github.com/users/saad-ali/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saad-ali/subscriptions","organizations_url":"https://api.github.com/users/saad-ali/orgs","repos_url":"https://api.github.com/users/saad-ali/repos","events_url":"https://api.github.com/users/saad-ali/events{/privacy}","received_events_url":"https://api.github.com/users/saad-ali/received_events","type":"User","site_admin":false},"open_issues":328,"closed_issues":385,"state":"open","created_at":"2016-11-12T00:35:38Z","updated_at":"2017-02-26T10:36:36Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2017-02-24T17:15:59Z","updated_at":"2017-02-26T11:06:20Z","closed_at":null,"body":"Version:\r\nServer Version: version.Info{Major:\"1\", Minor:\"7+\", GitVersion:\"v1.7.0-alpha.0.190+2bb97435eb5af0\", GitCommit:\"2bb9\r\n7435eb5af02c6e75964c909069e438b19bae\", GitTreeState:\"clean\", BuildDate:\"2017-02-24T13:41:51Z\", GoVersion:\"go1.7.5\",\r\n Compiler:\"gc\", Platform:\"linux/amd64\"} - HEAD, synced today.\r\n\r\nI started K8S cluster with ./cluster/kube-up.sh with Cluster Autoscaler enabled. I modified the CA manifest file on the master. In logs the new autoscaler complained about port 8085 being used (by the old CA) and the old one stopped writing logs COUPLE seconds after this port unavailable entry was written. \r\n\r\nI guess the old POD should be stopped first and only then the new pod should started.\r\n\r\ncc: @kubernetes/sig-node-bugs @dchen1107 @vishh @yujuhong  @fgrzadkowski @MaciekPytel\r\n"},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/282548330","html_url":"https://github.com/kubernetes/kubernetes/issues/42061#issuecomment-282548330","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/42061","id":282548330,"user":{"login":"wojtek-t","id":10743879,"avatar_url":"https://avatars.githubusercontent.com/u/10743879?v=3","gravatar_id":"","url":"https://api.github.com/users/wojtek-t","html_url":"https://github.com/wojtek-t","followers_url":"https://api.github.com/users/wojtek-t/followers","following_url":"https://api.github.com/users/wojtek-t/following{/other_user}","gists_url":"https://api.github.com/users/wojtek-t/gists{/gist_id}","starred_url":"https://api.github.com/users/wojtek-t/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wojtek-t/subscriptions","organizations_url":"https://api.github.com/users/wojtek-t/orgs","repos_url":"https://api.github.com/users/wojtek-t/repos","events_url":"https://api.github.com/users/wojtek-t/events{/privacy}","received_events_url":"https://api.github.com/users/wojtek-t/received_events","type":"User","site_admin":false},"created_at":"2017-02-26T11:06:20Z","updated_at":"2017-02-26T11:06:20Z","body":"> In your use case, why are you using static pod for Cluster Autoscaler? Why not using daemonset? If you must use the static pod, please introducing your own controller to guarantee the order between old and new pods. Same story for etcd server too. cc/ @mml who is working on etcd3 upgrade.\r\n\r\nAs for \"etcd\" - how you can make it a DaemonSet? If etcd is not running, apiserver is not running, and thus any other components won't know what they should be scheduling/creating/running. This just won't work.\r\nIn a setup with a single master instance (or to be more clear: with a single etcd instance), this is just undoable.\r\n\r\nAs for cluster autoscaler, this would work as long as master node is registered in the cluster. Which is not the case in GKE, which means we can't really do it now.\r\n\r\n> I am inclined to close this issue as the design intended.\r\n\r\nI was really surprised that this is by-design, but reading your rationale it make sense to me. However I think should be very clearly documented, and I wasn't able to find it anywhere. So we should probably at least document it before closng this issue."}}
{"actor":{"display_login":"wojtek-t"}}