Skip to content

Conversation

grolu
Copy link
Contributor

@grolu grolu commented Jul 3, 2025

What this PR does / why we need it:

Screenshot 2025-07-03 at 17 14 39

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Shoot constraints indicating ongoing operations are now correctly rendered with a progress indicator and the status label 'Operation in Progress'

Make DualStackNodesMigrationReady a progress constraint
@gardener-robot gardener-robot added needs/review Needs review size/s Size of pull request is small (see gardener-robot robot/bots/size.py) labels Jul 3, 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 Jul 3, 2025
@gardener-robot-ci-1 gardener-robot-ci-1 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 Jul 3, 2025
@grolu
Copy link
Contributor Author

grolu commented Jul 3, 2025

@axel7born

@axel7born
Copy link
Contributor

Perfect!
Thanks a lot!

@gardener-robot
Copy link

@holgerkoser, @klocke-io You have pull request review open invite, please check

- Always return actual status text
- Overwrite status text in configuration
- Map "Progressing" (for unhealthy conditions) to "Degraded"
@gardener-robot gardener-robot added size/m Size of pull request is medium (see gardener-robot robot/bots/size.py) and removed size/s Size of pull request is small (see gardener-robot robot/bots/size.py) labels Jul 7, 2025
@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 Jul 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 Jul 7, 2025
@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 Jul 7, 2025
Copy link
Contributor

@klocke-io klocke-io left a comment

Choose a reason for hiding this comment

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

🚀

@grolu grolu marked this pull request as draft August 19, 2025 08:47
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/review Needs review size/m Size of pull request is medium (see gardener-robot robot/bots/size.py)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants