Skip to content

Conversation

@britaniar
Copy link
Collaborator

@britaniar britaniar commented Dec 10, 2025

Description of your changes

I have:

  • Added a check to make sure no cluster start updating and let currently updating cluster finish within a stage before marking the stage and update run as stopped.

  • Update the stage condition status as stopping or stopped.

  • Update integration tests and added UTs.

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

  • Integration Test
  • Unit Test

Special notes for your reviewer

While waiting for cluster to finish updating, stage and update run will have a progressing unknown condition with the reason as stopping.

@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

❌ Patch coverage is 83.57143% with 23 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/controllers/updaterun/stop.go 83.89% 13 Missing and 6 partials ⚠️
pkg/controllers/updaterun/controller.go 76.47% 2 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
@britaniar britaniar marked this pull request as ready for review December 11, 2025 17:23
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants