search

operator-framework / operator-controller

A new and improved management framework for extending Kubernetes with Operators
https://operator-framework.github.io/operator-controller/
Apache License 2.0
54 stars 53 forks source link

Review Reason and Type strings in ClusterExtension API. #880

Open tmshort opened 4 months ago

tmshort commented 4 months ago

We discussed a similar topic here: do we really need these reasons? I don't think we have a use case for this just yet.

ReasonErrorGettingClient can be replaced with a more generic ReasonInstallationFailed. Same, probably, goes for ReasonBundleLoadFailed too, but there is another question about related condition type TypeHasValidBundle above.

See also #842

_Originally posted by @m1kola in https://github.com/operator-framework/operator-controller/pull/846#discussion_r1604761276_

tmshort commented 4 months ago

Looks as though #842 changed title, so it's not just making conditions consistent any more. This should probably be part of the effort of this issue.

tmshort commented 3 months ago

@bentito are you basically working on this?

bentito commented 3 months ago

yes