-
Notifications
You must be signed in to change notification settings - Fork 18
Description
Enhancement
Why is this needed?:
Latest Kubernetes API conventions highly recommend status conditions
- https://book.kubebuilder.io/reference/good-practices#why-you-should-adopt-status-conditions
- https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api-conventions.md#typical-status-properties
Particularly, boolean statuses like the planned ReadyToUse are strongly advised against, with conditions being the recommended replacement.
Similarly, advice is that error statuses like TimestampedError don't age well, and conditions are recommended as an alternative.
COSI should draft a plan for what conditions to have, and if they look good, the v1alpha2 KEP will be updated. IMO, it should be possible for the v1a2 KEP to be approved with the current API and to make these changes here, which will need to be more formally okayed for v1beta1.
Describe the solution you'd like in detail:
For the v1alpha2 implementation, COSI should draft a plan for using conditions instead of these 2 status patterns.
Describe alternatives you've considered:
Alternatives considered are the ReadyToUse bool and TimestampedError inherited as recommendations from the Volume Snapshot project.
Other alternative considerations haven't been considered as of Dec 2025.
Additional context:
This came up in API review here: #199