Skip to content

[DRAFT] Support "Progress Constraints"#2503

Draft
grolu wants to merge 3 commits intomasterfrom
enh/progress-constraints
Draft

[DRAFT] Support "Progress Constraints"#2503
grolu wants to merge 3 commits intomasterfrom
enh/progress-constraints

Conversation

@grolu
Copy link
Member

@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'

@grolu
Support "Progress Constraints"
947ab70 
Make DualStackNodesMigrationReady a progress constraint
@gardener-robot gardener-robot added needs/review Needs review size/S Denotes a PR that changes 10-29 lines, ignoring generated files. 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
Member Author

grolu commented Jul 3, 2025

@axel7born

@axel7born
Copy link
Contributor

axel7born commented Jul 3, 2025

Perfect!
Thanks a lot!

@gardener-robot
Copy link

gardener-robot commented Jul 6, 2025

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

@grolu
Updated StatusTags
857808c 
- Always return actual status text
- Overwrite status text in configuration
- Map "Progressing" (for unhealthy conditions) to "Degraded"
@gardener-robot gardener-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. 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
@axel7born axel7born mentioned this pull request Jul 7, 2025
Merged
@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
klocke-io
klocke-io approved these changes Aug 8, 2025
View reviewed changes
Copy link
Member

@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
@grolu grolu added the area/ipcei IPCEI (Important Project of Common European Interest) label Sep 25, 2025
@grolu grolu mentioned this pull request Sep 26, 2025
Open
10 tasks
@grolu grolu changed the title Support "Progress Constraints" [DRAFT] Support "Progress Constraints" Oct 1, 2025
@grolu grolu mentioned this pull request Oct 16, 2025
Open
@gardener-ci-robot
Copy link
Contributor

gardener-ci-robot commented Dec 19, 2025

The Gardener project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

  • After 15d of inactivity, lifecycle/stale is applied
  • After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Mark this PR as rotten with /lifecycle rotten
  • Close this PR with /close

/lifecycle stale

@gardener-robot
Copy link

gardener-robot commented Dec 19, 2025

@gardener-ci-robot Command /add is not available to you but only to a Maintainer, Member, Author.

@gardener-ci-robot
Copy link
Contributor

gardener-ci-robot commented Jan 3, 2026

The Gardener project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

  • After 15d of inactivity, lifecycle/stale is applied
  • After 15d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 7d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Mark this PR as rotten with /lifecycle rotten
  • Close this PR with /close

/lifecycle stale

@gardener-robot gardener-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 3, 2026
@gardener-ci-robot
Copy link
Contributor

gardener-ci-robot commented Feb 2, 2026

The Gardener project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:

  • After 30d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 14d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as active with /lifecycle active
  • Mark this PR as fresh with /remove-lifecycle rotten
  • Close this PR with /close

/lifecycle rotten

@gardener-prow gardener-prow bot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Feb 2, 2026
@grolu grolu added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. labels Feb 3, 2026
@gardener-prow gardener-prow bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 3, 2026
@gardener-prow
Copy link

gardener-prow bot commented Mar 3, 2026

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@gardener-prow gardener-prow bot added the cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. label Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@klocke-io klocke-io klocke-io approved these changes

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

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

Assignees

No one assigned

Labels

area/ipcei IPCEI (Important Project of Common European Interest) cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. 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-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@grolu @axel7born @gardener-robot @gardener-ci-robot @klocke-io @gardener-robot-ci-1 @gardener-robot-ci-2