{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/origin/issues/5310","labels_url":"https://api.github.com/repos/openshift/origin/issues/5310/labels{/name}","comments_url":"https://api.github.com/repos/openshift/origin/issues/5310/comments","events_url":"https://api.github.com/repos/openshift/origin/issues/5310/events","html_url":"https://github.com/openshift/origin/pull/5310","id":112704320,"number":5310,"title":"Enable HorizontalPodAutoscaler Support for DeploymentConfigs","user":{"login":"DirectXMan12","id":119737,"avatar_url":"https://avatars.githubusercontent.com/u/119737?v=3","gravatar_id":"","url":"https://api.github.com/users/DirectXMan12","html_url":"https://github.com/DirectXMan12","followers_url":"https://api.github.com/users/DirectXMan12/followers","following_url":"https://api.github.com/users/DirectXMan12/following{/other_user}","gists_url":"https://api.github.com/users/DirectXMan12/gists{/gist_id}","starred_url":"https://api.github.com/users/DirectXMan12/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DirectXMan12/subscriptions","organizations_url":"https://api.github.com/users/DirectXMan12/orgs","repos_url":"https://api.github.com/users/DirectXMan12/repos","events_url":"https://api.github.com/users/DirectXMan12/events{/privacy}","received_events_url":"https://api.github.com/users/DirectXMan12/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openshift/origin/labels/approved","name":"approved","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"deads2k","id":8225098,"avatar_url":"https://avatars.githubusercontent.com/u/8225098?v=3","gravatar_id":"","url":"https://api.github.com/users/deads2k","html_url":"https://github.com/deads2k","followers_url":"https://api.github.com/users/deads2k/followers","following_url":"https://api.github.com/users/deads2k/following{/other_user}","gists_url":"https://api.github.com/users/deads2k/gists{/gist_id}","starred_url":"https://api.github.com/users/deads2k/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deads2k/subscriptions","organizations_url":"https://api.github.com/users/deads2k/orgs","repos_url":"https://api.github.com/users/deads2k/repos","events_url":"https://api.github.com/users/deads2k/events{/privacy}","received_events_url":"https://api.github.com/users/deads2k/received_events","type":"User","site_admin":false},"milestone":null,"comments":44,"created_at":"2015-10-22T00:16:33Z","updated_at":"2015-11-02T20:41:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openshift/origin/pulls/5310","html_url":"https://github.com/openshift/origin/pull/5310","diff_url":"https://github.com/openshift/origin/pull/5310.diff","patch_url":"https://github.com/openshift/origin/pull/5310.patch"},"body":"This PR adds support for HPAs scaling DeploymentConfigs.   A new \"HybridClient\" struct is introduced to support Namespacers that need to point at resources or subresources that could be in either the Origin API or the Kube API (like `Scale`).\r\n\r\nThis commit also imports the Scale kind into the Origin v1 API (but it's still \"backed\" in the unversioned API by the underlying Kube resource).\r\n\r\nFinally, this commit contains a patch to Kube that converts the HPA controller from using a single `Client` type to using separate `Namespacers` for its various needs.\r\n\r\n\r\n--- \r\nTODOs\r\n - [x] cherry-pick kubernetes/kubernetes#16570 and update policy.\r\n - [x] cherry pick kubernetes/kubernetes#15706 from upstream into this PR (underneath your changes\r\n - [x] open issue for \" fix the metrics client to take pod names, rather than requiring cluster-wide pod-listing privileges\" https://github.com/openshift/origin/pull/5310/files#r43427148 (DONE: kubernetes/kubernetes#16675)\r\n - [x] rename UPSTREAM commits to match `UPSTREAM: PR` or `UPSTREAM: <carry>` format\r\n - [x] remove WIP from commit name\r\n - [ ] open issue for \" fix the client to no longer need root proxy access\" https://github.com/openshift/origin/pull/5310/files#r43427078\r\n - [ ] add end-to-end or extended test in origin to prove that this works without default (and comparatively restrictive) policy.\r\n - [x] add a comment to `Update` to document the order in which the deploymentconfig controller does the updates to resources as it creates new RCs and changes its RC's replicas.  That information is required to reason about race potential.\r\n - [x] resolve issue with cross-namespace HPA requests (prevent it and add a test ensuring it is prevented) https://github.com/openshift/origin/pull/5579"},"comment":{"url":"https://api.github.com/repos/openshift/origin/issues/comments/153150168","html_url":"https://github.com/openshift/origin/pull/5310#issuecomment-153150168","issue_url":"https://api.github.com/repos/openshift/origin/issues/5310","id":153150168,"user":{"login":"openshift-bot","id":1779249,"avatar_url":"https://avatars.githubusercontent.com/u/1779249?v=3","gravatar_id":"","url":"https://api.github.com/users/openshift-bot","html_url":"https://github.com/openshift-bot","followers_url":"https://api.github.com/users/openshift-bot/followers","following_url":"https://api.github.com/users/openshift-bot/following{/other_user}","gists_url":"https://api.github.com/users/openshift-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/openshift-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openshift-bot/subscriptions","organizations_url":"https://api.github.com/users/openshift-bot/orgs","repos_url":"https://api.github.com/users/openshift-bot/repos","events_url":"https://api.github.com/users/openshift-bot/events{/privacy}","received_events_url":"https://api.github.com/users/openshift-bot/received_events","type":"User","site_admin":false},"created_at":"2015-11-02T20:41:16Z","updated_at":"2015-11-02T20:41:16Z","body":"continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/6604/)"}}