Skip to content

Conversation

@WIZARD-CXY
Copy link
Contributor

@WIZARD-CXY WIZARD-CXY commented Sep 6, 2017

Add a sentence in the hpa doc to explicitly specify that hpa doesn't support daemonset to fix kubernetes/kubernetes#48588


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 6, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Sep 6, 2017

Deploy preview ready!

Built with commit f520d39

https://deploy-preview-5313--kubernetes-io-master-staging.netlify.com

in a replication controller, deployment or replica set based on observed CPU utilization
(or, with alpha support, on some other, application-provided metrics).
(or, with alpha support, on some other, application-provided metrics). Note that Horizontal
Pod Autoscaling does not support object that can't be scaled like DaemonSet.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe: ... Pod Autoscaling does not apply to objects that can't be scaled, for example, DaemonSet.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

object -> resources ?

@WIZARD-CXY
Copy link
Contributor Author

Thanks for the review, @resouer changed according to your suggestion PTAL.

@resouer
Copy link
Contributor

resouer commented Sep 6, 2017

/lgtm

@resouer resouer merged commit 75a104c into kubernetes:master Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Should not support HPA for DaemonSet

6 participants