cluster-api icon indicating copy to clipboard operation
cluster-api copied to clipboard

✨ migrate Requeue to RequeueAfter

Open sivchari opened this issue 6 months ago • 5 comments

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #12272

sivchari avatar May 29 '25 11:05 sivchari

This PR is currently missing an area label, which is used to identify the modified component when generating release notes.

Area labels can be added by org members by writing /area ${COMPONENT} in a comment

Please see the labels list for possible areas.

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.

k8s-ci-robot avatar May 29 '25 11:05 k8s-ci-robot

/assign

I don't know when I'll get around to it, but I want to review this PR before it's merged

sbueringer avatar May 30 '25 08:05 sbueringer

Will try to get back to this once we're done with v1beta2 for this release

sbueringer avatar Jun 12 '25 10:06 sbueringer

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please ask for approval from sbueringer. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Jun 12 '25 10:06 k8s-ci-robot

/retest

sivchari avatar Jun 13 '25 00:06 sivchari

PR needs rebase.

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.

k8s-ci-robot avatar Sep 15 '25 18:09 k8s-ci-robot

@sivchari

Can we split this up in separate PRs? One PR after another per reconciler would be good.

Some of these changes are non-trivial (as we have to access the impact on the reconcile behavior case by case) and breaking this down into smaller PRs makes it more realistic that we find enough time for reviews (and don't have to rebase too often)

sbueringer avatar Nov 11 '25 17:11 sbueringer

@sbueringer Sure, I close this PR once, and then I restart to work on it.

sivchari avatar Nov 12 '25 02:11 sivchari

/close

sivchari avatar Nov 12 '25 02:11 sivchari

@sivchari: Closed this PR.

In response to this:

/close

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.

k8s-ci-robot avatar Nov 12 '25 02:11 k8s-ci-robot