Nic Cope

Results 846 comments of Nic Cope

@ulucinar I agree we should update both. https://github.com/crossplane/crossplane-runtime/issues/285 might block that though - I'm currently looking into it.

I'd prefer to start simple here, at least until we have more confidence that folks would benefit from different poll intervals for different individual managed resources (or kinds of managed...

@srueg It does seem like there is a subtle difference here. Specifically: 1. `SyncPeriod` seems to work by setting the `ResyncPeriod` of the controller-runtime's cache. The cache is a bunch...

Intuitively `Synced=False` seems correct to me. A resource that is paused may not not have its desired state synced, and a resource that is experiencing a reconcile error may not...

I'm not sure how much we can really change at this point - I would consider status conditions as part of a resource's API and thus changing the status conditions...

@captainroy-hy Thanks for opening this! I'm not aware of anyone planning to work on this, but the functionality is still desirable. Contributions are definitely welcome. I will warn though that...

I think controller-runtime's strategy seems reasonable. My straw-man proposal to get this started would be to adopt a modified version of controller-runtime that: * Uses Github labels instead of emojis...

> Uses Github labels instead of emojis to categorise PRs I just started working on updating the controller-runtime docs for our needs, and realised that folks without at least "triage"...

@hasheddan Any thoughts on how we'd present an aggregated array of errors cleanly? iirc there's a couple of different places in which we'd return these - e.g. `kubectl crossplane`, revision...