Skip to content

Conversation

gagan16k
Copy link
Member

@gagan16k gagan16k commented Aug 28, 2025

What this PR does / why we need it:
This PR introduces log messages to indicate when rollout operations are ongoing for MachineDeployments and when nodes are being annotated with cluster autoscaler scale-down annotations.
Changes made for InPlace, Rolling and Recreate

Special notes for your reviewer:
Passed unit tests

Release note:

Add informational logging for MachineDeployment rollout events

@gagan16k gagan16k requested a review from a team as a code owner August 28, 2025 03:47
@gardener-robot gardener-robot added the needs/review Needs review label Aug 28, 2025
@gagan16k gagan16k self-assigned this Aug 28, 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 28, 2025
@gagan16k gagan16k removed their assignment Aug 28, 2025
@gardener-robot gardener-robot added the size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) label Aug 28, 2025
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.

/lgtm

@gardener-robot gardener-robot added reviewed/lgtm Has approval for merging and removed needs/review Needs review labels Aug 29, 2025
@aaronfern aaronfern merged commit 3ce4da2 into gardener:master Aug 29, 2025
14 checks passed
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Aug 29, 2025
@gagan16k gagan16k deleted the rollout_log branch August 29, 2025 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewed/lgtm Has approval for merging reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants