Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clusterapi: Add ephemeral disk capacity annotation for scale from zero #5416

Conversation

cnmcavoy
Copy link
Contributor

What type of PR is this?

/kind feature

What this PR does / why we need it:

Updates the clusterapi provider to support an additional capacity annotation for ephemeral disk capacity on MachineDeployments to supply information about the nodegroup shape.

Without an ephemeral disk capacity supplied, workloads that request ephemeral disk will not trigger a scale up of nodegroups with zero replicas as the ephemeral disk capacity defaults to zero bytes.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

Added support for supplying ephemeral disk capacity when scaling to and from zero nodes for the cluster autoscaler's Cluster API provider. Enabling this feature will require changes by the user, for instruction please see the Cluster API (clusterapi) provider README file in the autoscaler repository.

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:


@k8s-ci-robot k8s-ci-robot added kind/feature Categorizes issue or PR as related to a new feature. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 12, 2023
@mwielgus mwielgus requested a review from elmiko January 31, 2023 12:40
@elmiko
Copy link
Contributor

elmiko commented Jan 31, 2023

this is a nice addition, thanks @cnmcavoy . i'll spend some time reviewing this week.

@elmiko
Copy link
Contributor

elmiko commented Jan 31, 2023

i've created kubernetes-sigs/cluster-api#8036 to cover the docs update in cluster-api

Copy link
Contributor

@elmiko elmiko left a comment

Choose a reason for hiding this comment

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

this is a nice update and i think the additional annotation is not controversial for the cluster-api community. thanks again @cnmcavoy

/approve
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 2, 2023
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cnmcavoy, elmiko

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 2, 2023
@k8s-ci-robot k8s-ci-robot merged commit c254ee3 into kubernetes:master Feb 2, 2023
k8s-ci-robot pushed a commit to kubernetes-sigs/cluster-api that referenced this pull request Feb 22, 2023
to include latest changes to the annotation that are proposed in the
cluster autoscaler.

ref: kubernetes/autoscaler#5382
ref: kubernetes/autoscaler#5416
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/cluster-autoscaler cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants