Terje Lafton
Terje Lafton
Sounds good! Implementing `kstatus` without any breaking changes would be perfect. I did however comment one concern [here](https://github.com/crossplane/crossplane/pull/5453#issuecomment-2074718148), but it's not the most relevant for this issue. I have mostly...
The new field name reads a lot better though and matches more with how I think about it! The desired way of doing this in Protobuf would be to create...
> @TerjeLafton In Crossplane, the function is the server and Crossplane is the client. If we remove and reserve the current field numbers, a Crossplane built after the change wouldn't...
Yes, it would work like that. When the `external.Update` function returns an error, we can just add a `xpv1.Updating()` condition to the `SetConditions` call. This would set the resource as...
Ready for review 😄 @phisco @pedjak @turkenh, long since I made the issue and promised this PR. It's been busy 😅
> Apologies for the delayed review, this fell out of my GitHub notifications somehow. No problem! Thanks for the review 😄 It couldn't be merged before the next version anyway,...
> @jbw976 and @TerjeLafton I've added more details in a [draft PR](https://github.com/crossplane/docs/pull/752). > > **Preview:** https://deploy-preview-752--crossplane.netlify.app/master/concepts/managed-resources/#conditions > > If you're good with this I'll add the same content to the...
@negz @turkenh @phisco As discussed in the PR, here is an attempt at a one-pager. Any review is greatly appreciated :)
Just a friendly reminder, @negz, in case it got lost in other tasks 😄
> @TerjeLafton Here's a quick brain dump of some things that I'd like to see discussed/considered in this design. Apologies if some of this is already covered, this is a...