Skip to content

Expose builder's ForOptions for a reconciler#676

Merged
scothis merged 1 commit intoreconcilerio:mainfrom
scothis:for-options
Feb 4, 2026
Merged

Expose builder's ForOptions for a reconciler#676
scothis merged 1 commit intoreconcilerio:mainfrom
scothis:for-options

Conversation

@scothis
Copy link
Member

@scothis scothis commented Feb 4, 2026

The builder's For method is called internally for ResourceReconciler and AggregateReconciler. It can only be called once and is the only way to configure some aspects of the controller.

The exposed options can now be defined on the reconciler as SetupForOptions, which are passed to the builder For method.

The builder's For method is called internally for ResourceReconciler and
AggregateReconciler. It can only be called once and is the only way to
configure some aspects of the controller.

The exposed options can now be defined on the reconciler as
SetupForOptions, which are passed to the builder For method.

Signed-off-by: Scott Andrews <scott@andrews.me>
@scothis
Copy link
Member Author

scothis commented Feb 4, 2026

Alternative to #673

cc @annismckenzie

@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.25%. Comparing base (5c1e9d9) to head (3da3af3).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
reconcilers/aggregate.go 0.00% 2 Missing ⚠️
reconcilers/resource.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #676   +/-   ##
=======================================
  Coverage   60.25%   60.25%           
=======================================
  Files          39       39           
  Lines        3628     3628           
=======================================
  Hits         2186     2186           
  Misses       1329     1329           
  Partials      113      113           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@scothis scothis merged commit 9c9ed61 into reconcilerio:main Feb 4, 2026
3 of 4 checks passed
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.

1 participant