Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@thiyyakat
Copy link
Member

@thiyyakat thiyyakat commented Jul 21, 2025

What this PR does / why we need it:

  • Sync with upstream autoscaler v1.33.0
  • Update debian base image to Debian 12

Which issue(s) this PR fixes:
Fixes #351, partially fixes MCM Issue #982

Special notes for your reviewer:

Release note:

Synced changes till v1.33.0 of upstream autoscaler and updated debian base image to Debian 12.

voelzmo and others added 30 commits April 7, 2025 17:15
Improve resilience of diskTypes requests.
Remove outdated GCE cloudprovider owners
…intsTesting

Add unit test for cleaning deletion soft taint in scale down cool down
Also consider containerStatus.resources when handling container resources
…y-sts-to-drain

Allow draining when StatefulSet kind has custom API Group
…ller

This commit instruments the `observer`, used within the `recommender`
e2e tests, reuse the test framework namespace. The main reason behind
this change is to allow running the test suite against an existing
cluster that already has `VPA` resources in other namespaces.

Currently, the test is considering all `namespaces` when it builds it's
`observer`'s controller, which makes the test fail when trying to validate
the recommendations for components outside of the test scope.

This behavior was discovered while running the e2e tests against a
cluster with VPA setup that uses Prometheus as history storage
provider.
…bserver-reuse-test-framework-namespace

e2e(recommender): Restrict the internal `VPA` controller scope to test namespace
…-metric

Emit scale down metric even when there is no scale down candidates.
…down-signature

Patch TestCleaningSoftTaintsInScaleDown to be compatible with new isScaleDownInCooldown signature
@gardener-robot-ci-3 gardener-robot-ci-3 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 Jul 31, 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 Jul 31, 2025
@thiyyakat thiyyakat requested a review from aaronfern July 31, 2025 05:10
@gardener-robot-ci-1 gardener-robot-ci-1 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 Jul 31, 2025
@gardener-robot
Copy link

@unmarshall You have pull request review open invite, please check

Copy link
Member

@aaronfern aaronfern left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!
/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/rebase Needs git rebase needs/review Needs review needs/second-opinion Needs second review by someone else labels Aug 7, 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 Aug 7, 2025
Co-authored-by: Aaron Francis Fernandes <[email protected]>
@gardener-robot gardener-robot added needs/second-opinion Needs second review by someone else and removed reviewed/lgtm Has approval for merging labels Aug 14, 2025
@gardener-robot-ci-1 gardener-robot-ci-1 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 Aug 14, 2025
@gardener-robot-ci-2 gardener-robot-ci-2 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 Aug 14, 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 Aug 14, 2025
@aaronfern aaronfern merged commit 0fdfec8 into gardener:machine-controller-manager-provider Aug 14, 2025
10 of 11 checks passed
@gardener-robot gardener-robot added status/closed Issue is closed (either delivered or triaged) and removed status/accepted Issue was accepted as something we need to work on labels Aug 14, 2025
thiyyakat added a commit to thiyyakat/autoscaler that referenced this pull request Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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/second-opinion Needs second review by someone else size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue for - Sync with Upstream v1.33