Skip to content

Conversation

Diaphteiros
Copy link
Contributor

@Diaphteiros Diaphteiros commented Oct 2, 2025

What this PR does / why we need it:
Adds the possibility to specify functions that can overwrite the smart requeue value.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
Will be helpful in fixing a bug in the PlatformService MCP.

The PR contains a release.

Release note:

The status updater's `WithSmartRequeue` method now takes functions as optional arguments. These functions take the `ReconcileResult` and return a smart requeue value, which will then overwrite the value from the `ReconcileResult`. This allows determining the smart requeue value based on the object's updated conditions, which was complicated to achieve before this change.

@reshnm reshnm merged commit 57cba28 into main Oct 7, 2025
9 checks passed
@reshnm reshnm deleted the conditional-smartrequeue branch October 7, 2025 14:19
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.

2 participants