wmedvede commented on PR #372: URL: https://github.com/apache/incubator-kie-kogito-serverless-operator/pull/372#issuecomment-1936407277
@jordigilh @ricardozanini Let me add some positive feedback: In the usecase below, we are using a SFP with a global persistence configuration that we can see here: []([](https://github.com/flows-examples/techpreview2/tree/main/platforms/openshift-dataindex_and_jobservice_as_platform_service_and_global_persistence_postgresql)) Readme here: https://github.com/flows-examples/techpreview2/blob/main/platforms/openshift-dataindex_and_jobservice_as_platform_service_and_global_persistence_postgresql/Readme.txt **Summary:** 1. **SPF looks like this, and we expect DI and JS to use that configuration:** ``` persistence: postgresql: secretRef: name: postgres-secrets userKey: POSTGRESQL_USER passwordKey: POSTGRESQL_PASSWORD serviceRef: name: postgres databaseName: sonataflow2 ``` 2. **DI and JS started ok, and both services use their own schema as expected. Good man! @jordigilh** 3. **callbackstatetimeouts WF has started and worked ok, with a persistence configuration like this Good man!** ``` apiVersion: sonataflow.org/v1alpha08 kind: SonataFlow metadata: name: callbackstatetimeouts annotations: sonataflow.org/description: Callback State Timeouts Example k8s sonataflow.org/version: 0.0.1 spec: persistence: {} flow: start: PrintStartMessage ``` 4. **workflowtimeouts WF, with the configuration below has failed to deploy:** Here we would expect that the WF picks the platform persistence configuration and use it's own configured schema wf_timeouts. But the following error was produced at deployment. (Please @jordigilh correct me if the configuration is wrong. `The SonataFlow "workflowtimeouts" is invalid: spec.persistence.postgresql: Invalid value: 1: spec.persistence.postgresql in body should have at least 2 properties` ``` apiVersion: sonataflow.org/v1alpha08 kind: SonataFlow metadata: name: workflowtimeouts annotations: sonataflow.org/description: Workflow Timeouts Example k8s sonataflow.org/version: 0.0.1 spec: persistence: postgresql: serviceRef: databaseSchema: wf_timeouts ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
