Dr. Stefan Schimanski
Dr. Stefan Schimanski
Sorry for early review. This somehow hit my timeline 😀 I figure this is super early. Let me know when it's ready for a real review.
@stevekuznetsov no, projections cannot solve this. Projections are for all-or-nothing cases, i.e. to redirect a complete GVR to another service, a bit like aggregation in Kube, but hopefully without the...
No concerns to add a marker. It is not used today by Kube, but added just for completion as OpenAPI has it. Note that afaik it is not exposed in...
this reminds me of the upstream discussion about finegrained metadata field permissions, e.g. to add finalizers. cc @s-urbaniak
Is this actually helping? `t.Helper()` just removes one frame of the stack. I.e. you see the file+line of the parent (or some ancestor) during execution of the closure returned by...
I see that you comment exactly that in https://github.com/crossplane/crossplane/pull/5722#issuecomment-2148427387. So this PR is better than nothing clearly.
Have not read all the controller changes. Have you changed it to not have the shadow logical cluster? I.e. a mount is a mount, always. There is not this hybrid...
xref @p0lyn0mial's https://github.com/kubernetes-sigs/controller-runtime/pull/2934/files where he proved that controller-runtime can adopt watchlist.
@cupakob is this blocked on anything? Anything we can help with?