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

Backport patch for early abort on ASGs to cluster autoscaler 1.18.3 #4492

Conversation

drmorr0
Copy link
Contributor

@drmorr0 drmorr0 commented Nov 30, 2021

Upstream PR: #4489

feiskyer and others added 30 commits March 23, 2020 06:23
1.18: Ensure only capacity is changed when PUT Azure VMSS
Properly propagate scale-up failures in scale sets.
CA 1.18: Ensure VMSS is not under updating before scaling out
CA 1.18: Remove checking for VMSS provisioningState before scaling
Cherry-pick kubernetes#3036: Proactively decrement scale set count during deletion operations
Cherry-pick kubernetes#3019: Correct cloudprovider/azure's GPULabel to "accelerator"
…API version.

This change adds detection for an environment variable to specify the group for the clusterapi resources. If the environment
variable `CAPI_GROUP` is specified, then it will
be used instead of the default.
This also decouples the API group from the version and let the latter to be discovered dynamically.
[CA-1.18] kubernetes#2934 cherry-pick: Fixes 2932: let the capi version to be discovered
[CA-1.18] kubernetes#2940 cherry-pick CAPI: Stop panicking in newMachineController
Cherry-pick kubernetes#3141: Avoid sending extra deletion calls for in-progress deletions
[CA-1.18] kubernetes#2983 cherry-pick: ClusterAPI Provider: Provide fake proivder IDs for failed Machines
k8s-ci-robot and others added 21 commits October 5, 2020 09:53
Fix priority expander falling back to random although higher priority matches
Cherry pick 3308 onto 1.18 - Fix priority expander falling back to random although higher priority matches
…r-delete-fix

Remove ScaleDownNodeDeleted status since we no longer delete nodes synchronously
Improve Cluster API tests to work better with constrained resources
Merge pull request kubernetes#3570 from towca/jtuznik/scale-down-after-delete-fix
…32-1.18

Cherry-pick kubernetes#3532 onto 1.18: Azure: support allocatable resources overrides via VMSS tags
Signed-off-by: Marques Johansson <[email protected]>
[cluster-autoscaler] Backport fixes for packet provider to release-1.18
Set different hostname label for upcoming nodes
--
make change backwards compatible with breaking schedulernodeinfo to
schedulerframework change according to kubernetes@73a5cdf
…launch-templates

cache ASG InstanceTypes for AWS
…patch-cluster-autoscaler-1.18.3-airbnb2

Backport kubernetes#4073 to 1.18.3
…up-no-capacity

early abort if aws node group no capacity
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 30, 2021
@k8s-ci-robot
Copy link
Contributor

@drmorr0: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@drmorr0 drmorr0 closed this Nov 30, 2021
@linux-foundation-easycla
Copy link

CLA Not Signed

@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Nov 30, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: drmorr0
To complete the pull request process, please assign feiskyer after the PR has been reviewed.
You can assign the PR to them by writing /assign @feiskyer in a comment when ready.

The full list of commands accepted by this bot can be found 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

@drmorr0
Copy link
Contributor Author

drmorr0 commented Nov 30, 2021

Whoops, not what I meant to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.