streamnative / pulsar-resources-operator

Apache License 2.0
34 stars 15 forks source link

[improve]: new a column "Policy Status" for resources #222

Closed ericsyh closed 3 months ago

ericsyh commented 3 months ago

Background

The current status for resources only have true or false two options and resource-operator will check all API actions work will make that resource as true. But in some cases, a topic can works for pubsub but sometimes the configuration API call may fail for example to enable the topic level replication but this fail won't block the topic pubsub usage.

Suggestion

So, we want to introduce a new status call the "warn" which defines the topic works for the create, permission setting API but fails on other topic policies API. The "warn" status means the topic can work for pubsub but other policies setting are failed so the client, app won't be impacted.

ericsyh commented 3 months ago

Update

After a discussion, we may consider another way to add a new column called the "Policy Status" which indicates to the topic, namespace policies API setting status.

NAME                                       RESOURCE_NAME                                          GENERATION   OBSERVED_GENERATION   READY   POLICY_STATUS

deploy-pulsar-entities-entities-0d34fb75   persistent://cc/cdr/cdr-rated-ha                       1            1                     True    True