search

apache / incubator-kie-kogito-serverless-operator

Kubernetes operator for SonataFlow
Apache License 2.0
15 stars 41 forks source link

Enhance error messages in SonataFlowPlatform CR status for knative integration #521

Open jianrongzhang89 opened 1 month ago

jianrongzhang89 commented 1 month ago

Description

With the implementation for the integration of SonataFlow Operator with Knative: https://github.com/apache/incubator-kie-kogito-serverless-operator/pull/467, the platform reconciliation logic will log an error message when: 1) the broker used for creating triggers is not created or available, or 2) the SinkBinding has not injected K_SINK into the Job Services deployment and the reconciliation will resume after the condition has been resolved.

This follow up issue is to allow the operator to update the SonataFlowPlatform CR status with the corresponding error messages to inform the user that an error has happened.

Implementation ideas

To minimize the impact on the existing logic, it is suggested to use existing condition type "Running" with a Reason like "WaitingKnativeEventing" instead of introducing a new condition type.

jianrongzhang89 commented 1 week ago

This is a follow up issue after https://github.com/apache/incubator-kie-kogito-serverless-operator/pull/467 is merged.