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.

Sign up for GitHub

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

Jump to bottom

Sync upstream v1.31.1 #339

Open
wants to merge 1,810 commits into
base: machine-controller-manager-provider
Choose a base branch
from
Open

Sync upstream v1.31.1 #339

wants to merge 1,810 commits into from

Conversation

aaronfern
Copy link

What this PR does / why we need it:
Sync with upstream autoscaler v1.31.1

Which issue(s) this PR fixes:
Fixes #320

Special notes for your reviewer:

Release note:

Synced changes till v1.31.1 of upstream autoscaler

k8s-ci-robot and others added 30 commits June 27, 2024 05:41
@k8s-ci-robot
Merge pull request kubernetes#6874 from kisieland/update-annotations
3d48578 
Update ProvReq annotations to new prefix
@k8s-ci-robot
Merge pull request kubernetes#6975 from nojnhuh/azure-e2e-test
bab68c4 
Update Azure cluster-autoscaler e2e cluster template
@azylinski
Fix: GCE FetchAvailableDiskTypes zones parsing
97419db
@k8s-ci-robot
Merge pull request kubernetes#6909 from ialidzhikov/enh/recommender-l…
d0441f7 
…ogging

vpa-recommender: Log object's namespace
@voelzmo
Remove unsupported parameter from admission-controller
b689df7
@k8s-ci-robot
Merge pull request kubernetes#6988 from voelzmo/patch-4
8cf630a 
Remove not yet supported parameter from admission-controller deployment
@nojnhuh
Add Azure cluster-autoscaler e2e test
fcba053
@k8s-ci-robot
Merge pull request kubernetes#6986 from azylinski/fix-gce-FetchAvaila…
8dd36c9 
…bleDiskTypes-parsing

Fix: GCE FetchAvailableDiskTypes zones parsing
@k8s-ci-robot
Merge pull request kubernetes#6989 from nojnhuh/azure-e2e-test
069aab7 
Add Azure cluster-autoscaler e2e test
@adrianmoisey
selectedNamespace contains a single namespace
e14db24
@mweibel
fix(clusterapi): HasInstance/FindMachine with namespace prefix
98f9489
@k8s-ci-robot
Merge pull request kubernetes#6863 from rrangith/azure-default-sizes
3c6dd26 
Default min/max sizes for Azure VMSSs
@nojnhuh
Update Azure cluster-autoscaler e2e test
999dc87
@k8s-ci-robot
Merge pull request kubernetes#6992 from nojnhuh/azure-e2e-test
965c84a 
Update Azure cluster-autoscaler e2e test
@robert-heinzmann-logmein
Fixing scale from zero if the nodeSelector of workload contains 'node…
b17f420 
….kubernetes.io/instance-type'
@robert-heinzmann-logmein
fix tests
9c18b5b
@nojnhuh
cluster-autoscaler Azure e2e: move Helm to ginkgo
3f18f06
@robert-heinzmann-logmein
fixing non flex shapes
5d61057
@morshielt
Change AnnotationNodeInfoProvider nested provider to interface
4260701
@ialidzhikov
vpa-updater: Add support for leader election
eeedaea
@ialidzhikov
vpa-recommender: Add support for leader election
211ecdc
@ialidzhikov
Fix the failing liveness probe for the non-leader replica
d5c12cf
@yaroslava-serdiuk
Do not fail if multiple ProvReqs are injected
2824fc9
@yaroslava-serdiuk @BigDarkClown
Update cluster-autoscaler/provisioningrequest/provreqclient/client.go
81b6d3d 
Co-authored-by: Bartek Wróblewski <[email protected]>
@yaroslava-serdiuk @BigDarkClown
Update cluster-autoscaler/provisioningrequest/provreqclient/client.go
1953031 
Co-authored-by: Bartek Wróblewski <[email protected]>
@k8s-ci-robot
Merge pull request kubernetes#6997 from morshielt/node_info_prov_iface
4d05113 
Change GCE AnnotationNodeInfoProvider nested provider to interface instead of concrete type
@adrianmoisey
Revert "Pass around comma separate strings rather than lists"
a7f2c93 
This reverts commit 7a1aea1.

As per
kubernetes#6788 (comment)
and discussion in DM. The preference is to split the string inside the
main.go file.
@adrianmoisey
Revert unintentional change
7e19ebe
@adrianmoisey
Remove hardcoded 10 second delay
edc8091 
Since tests will take 10 seconds on each pass to run
@k8s-ci-robot
Merge pull request kubernetes#6994 from nojnhuh/azure-e2e-test
bb5d064 
cluster-autoscaler Azure e2e: move Helm to ginkgo
@gardener-robot gardener-robot added needs/review Needs review size/xl Size of pull request is huge (see gardener-robot robot/bots/size.py) needs/second-opinion Needs second review by someone else labels Dec 3, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Dec 3, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Dec 3, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Dec 3, 2024
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Dec 3, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Dec 3, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Dec 3, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Dec 3, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Dec 9, 2024
@marc1404 marc1404 mentioned this pull request Feb 5, 2025
Open
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Mar 5, 2025
@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Mar 5, 2025
@gardener-robot-ci-3 gardener-robot-ci-3 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unmarshall unmarshall Awaiting requested review from unmarshall unmarshall is a code owner

@rishabh-11 rishabh-11 Awaiting requested review from rishabh-11

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/review Needs review needs/second-opinion Needs second review by someone else size/xl Size of pull request is huge (see gardener-robot robot/bots/size.py)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Issue for - Sync with Upstream v1.31