Sync upstream v1.29.2#302
Merged
rishabh-11 merged 1056 commits intogardener:machine-controller-manager-providerfrom May 21, 2024
Merged
Sync upstream v1.29.2#302rishabh-11 merged 1056 commits intogardener:machine-controller-manager-providerfrom
rishabh-11 merged 1056 commits intogardener:machine-controller-manager-providerfrom
Conversation
If the 'get' and 'list' verbs are present, but the 'watch' verb is absent, the autoscaler reports an error. For example: cluster-autoscaler-b8949d8b9-76vcd E1006 22:11:43.056176 1 reflector.go:148] k8s.io/client-go/dynamic/dynamicinformer/informer.go:108: Failed to watch infrastructure.cluster.x-k8s.io/v1beta2, Resource=vcdmachinetemplates: unknown
clusterapi: Add 'watch' verb to scale-from-zero example
autoscaler: fix premature end of binpacking
…vertical-pod-autoscaler/builder/golang-1.21.2 Bump golang from 1.21.1 to 1.21.2 in /vertical-pod-autoscaler/builder
feat(tencentcloud): remove dependency on tke interface
Updated the owners of externalgrpc cloudprovider
Azure: add ENV coverage to UT
… to drainability rules
…lity Convert scale-down checks to drainability rules
Update ec2_instance_types with make generate
- HasNodeGroupStartedScaleUp checks wheter a scale up request exists without checking any upcoming nodes.
Add HasNodeGroupStartedScaleUp to cluster state registry.
jbartosik et al are transitioning off of workload autoscalers (incl vpa and addon-resizer). kwiesmueller is on the new team and has agreed to take on reviewer/approver responsibilities.
Add kwiesmueller to OWNERS
…-fix Add information about provisioning-class-name annotation.
Fix multiple run of informers created in fetcher.go
gardener-robot-ci-1
added
the
reviewed/ok-to-test
gardener-robot-ci-2
removed
the
reviewed/ok-to-test
gardener-robot-ci-2
added
the
reviewed/ok-to-test
gardener-robot-ci-1
added
reviewed/ok-to-test
gardener-robot-ci-2
removed
the
reviewed/ok-to-test
aaronfern
approved these changes
May 10, 2024
Member
aaronfern
left a comment
aaronfern
left a comment
There was a problem hiding this comment.
Thanks for the changes!
/lgtm
/second-opinion @rishabh-11
gardener-robot
removed
needs/changes
gardener-robot-ci-2
added
the
reviewed/ok-to-test
gardener-robot-ci-1
removed
the
reviewed/ok-to-test
ghost
added
reviewed/ok-to-test
ghost
added
reviewed/ok-to-test
gardener-robot
added
reviewed/lgtm
gardener-robot-ci-2
added
the
reviewed/ok-to-test
gardener-robot
added
the
status/closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
Sync with upstream autoscaler
v1.29.2Which issue(s) this PR fixes:
Fixes #275
Special notes for your reviewer:
Release note: