-
Notifications
You must be signed in to change notification settings - Fork 18
Handle pointer items in ObjectList with ChildReconciler #480
Conversation
Canonically, Items in an ObjectList are a slice of structs. Some types, like Istio, use a slice of pointers intead. We now detect and handle both. Signed-off-by: Scott Andrews <scott@andrews.me>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #480 +/- ##
==========================================
+ Coverage 60.67% 61.02% +0.34%
==========================================
Files 26 27 +1
Lines 2543 2581 +38
==========================================
+ Hits 1543 1575 +32
- Misses 911 917 +6
Partials 89 89 ☔ View full report in Codecov by Sentry. |
@scothis, we have received your signed contributor license agreement. The review is usually completed within a week, but may take longer under certain circumstances. Another comment will be added to the pull request to notify you when the merge can proceed. |
@neowulf can you give this branch a try? |
Because personally I can never recall the correct incantation, this may help: go mod edit -replace github.com/vmware-labs/reconciler-runtime=github.com/scothis/reconciler-runtime@object-list-pointer
go mod tidy |
Signed-off-by: Scott Andrews <scott@andrews.me>
Thank you for the quick response! Yes, this patch works! |
@scothis, VMware has approved your signed contributor license agreement. |
Canonically, Items in an ObjectList are a slice of structs. Some types, like Istio, use a slice of pointers intead. We now detect and handle both.
Resolves #479