{"action":"created","issue":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/39295","repository_url":"https://api.github.com/repos/kubernetes/kubernetes","labels_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/39295/labels{/name}","comments_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/39295/comments","events_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/39295/events","html_url":"https://github.com/kubernetes/kubernetes/issues/39295","id":197962637,"number":39295,"title":"Confuse the rollout result after proportionally scale","user":{"login":"chengzhang1016","id":18393357,"avatar_url":"https://avatars.githubusercontent.com/u/18393357?v=3","gravatar_id":"","url":"https://api.github.com/users/chengzhang1016","html_url":"https://github.com/chengzhang1016","followers_url":"https://api.github.com/users/chengzhang1016/followers","following_url":"https://api.github.com/users/chengzhang1016/following{/other_user}","gists_url":"https://api.github.com/users/chengzhang1016/gists{/gist_id}","starred_url":"https://api.github.com/users/chengzhang1016/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chengzhang1016/subscriptions","organizations_url":"https://api.github.com/users/chengzhang1016/orgs","repos_url":"https://api.github.com/users/chengzhang1016/repos","events_url":"https://api.github.com/users/chengzhang1016/events{/privacy}","received_events_url":"https://api.github.com/users/chengzhang1016/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-12-29T06:56:35Z","updated_at":"2016-12-29T07:08:05Z","closed_at":null,"body":"Description of problem:\r\nI noticed you have new PR (controller: scale proportionally before rolling out new templates): https://github.com/kubernetes/kubernetes/pull/32995\r\nSo I did below test, and want to discuss with you for step 3 and step7. In another word, I want to know how to rollout after scale proportionally. How to select which RS to do scaling down when new RS was created?\r\n\r\nkubectl version: v1.6.0-alpha.0.2586+d4bf500e7340e6-dirty\r\ndocker version: 1.12.2-rc2\r\n\r\nSteps to Reproduce:\r\n1. Create a deployment with a RS: RS1=10\r\n```\r\napiVersion: extensions/v1beta1\r\nkind: Deployment\r\nmetadata:\r\n  name: hello-openshift\r\nspec:\r\n  replicas: 10\r\n  template:\r\n    metadata:\r\n      labels:\r\n        app: hello-openshift\r\n    spec:\r\n      containers:\r\n      - name: hello-openshift\r\n        image: openshift/hello-openshift\r\n        ports:\r\n        - containerPort: 80\r\n  strategy:\r\n    rollingUpdate:\r\n      maxSurge: 3\r\n      maxUnavailable: 2\r\n    type: RollingUpdate\r\n```\r\n\r\n2. Edit deployment ( change image to 'not-exist-1' )\r\nActual result:\r\nRS1=8\r\nRS2=5\r\n\r\n3. Edit deployment: Scale + Update image ( change replicas to 30 and image to 'not-exist-2')\r\nActual result:\r\nRS1=15\r\nRS2=13\r\nRS3=5\r\n\r\n4. Pausing the deployment\r\n\r\n5. Edit deployment: Scale + Update image ( change replicas to 30 and image to 'not-exist-3')\r\nActual result:\r\nRS1=24\r\nRS2=21\r\nRS3=8\r\n\r\n6. Resuming the deployment\r\n\r\n7. Check replicas of RSs\r\nActual result:\r\nSometimes are:\r\nRS1=24\r\nRS2=16\r\nRS3=8\r\nRS4=5\r\nSometimes are:\r\nRS1=19\r\nRS2=21\r\nRS3=8\r\nRS4=5\r\n\r\n\r\nAddition info: \r\nFollow your new PR 32995:\r\nIn Step 3: We actual did proportionally scale and then rolling out\r\n3.1 proportionally scale\r\nRS1=20\r\nRS2=13\r\n3.2 rolling out: RS1 scale down 5, RS3 scale up 5\r\nRS1=15\r\nRS2=13\r\nRS3=5\r\n\r\nQuestion: Why not is that RS2 scale down 5, RS3 scale up 5\r\nRS1=20\r\nRS2=8\r\nRS3=5\r\n\r\n\r\nIn Step 7: Sometimes have different result\r\nResult A: Caused by RS2 scale down 5, RS4 scale up 5\r\nRS1=24\r\nRS2=16\r\nRS3=8\r\nRS4=5\r\n\r\nResult B: Caused by RS1 scale down 5, RS4 scale up 5\r\nRS1=19\r\nRS2=21\r\nRS3=8\r\nRS4=5\r\n\r\nQuestion:Which one is correct? How to understanding the result?"},"comment":{"url":"https://api.github.com/repos/kubernetes/kubernetes/issues/comments/269590637","html_url":"https://github.com/kubernetes/kubernetes/issues/39295#issuecomment-269590637","issue_url":"https://api.github.com/repos/kubernetes/kubernetes/issues/39295","id":269590637,"user":{"login":"chengzhang1016","id":18393357,"avatar_url":"https://avatars.githubusercontent.com/u/18393357?v=3","gravatar_id":"","url":"https://api.github.com/users/chengzhang1016","html_url":"https://github.com/chengzhang1016","followers_url":"https://api.github.com/users/chengzhang1016/followers","following_url":"https://api.github.com/users/chengzhang1016/following{/other_user}","gists_url":"https://api.github.com/users/chengzhang1016/gists{/gist_id}","starred_url":"https://api.github.com/users/chengzhang1016/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chengzhang1016/subscriptions","organizations_url":"https://api.github.com/users/chengzhang1016/orgs","repos_url":"https://api.github.com/users/chengzhang1016/repos","events_url":"https://api.github.com/users/chengzhang1016/events{/privacy}","received_events_url":"https://api.github.com/users/chengzhang1016/received_events","type":"User","site_admin":false},"created_at":"2016-12-29T07:08:05Z","updated_at":"2016-12-29T07:08:05Z","body":"@mdshuai @zhouying7780 FYI."}}
{"actor":{"display_login":"chengzhang1016"}}