canonical / charmed-kubeflow-chisme

Shared Utilities used across Charmed Kubeflow
Apache License 2.0
1 stars 4 forks source link

fix `Prioritizer` to avoid setting `UnknownStatus` when empty #86

Closed ca-scribner closed 5 months ago

ca-scribner commented 5 months ago

The status prioritizer previously would report an UnknownStatus when it had no components to report statuses. UnknownStatus is a readonly status in Juju so this is not appropriate. This commit changes Prioritizer to identify when it has no components and instead return ActiveStatus (since a Prioritizer without anything is effectively Active).