cloudstateio / cloudstate

Distributed State Management for Serverless
https://cloudstate.io
Apache License 2.0
763 stars 97 forks source link

Deal with additional conditions properly #537

Closed marcoderama closed 2 years ago

marcoderama commented 3 years ago

These changes are a fix for Issue https://github.com/cloudstateio/cloudstate/issues/536.

The fundamental diffs are

After these changes, and testing with the setup of Issue https://github.com/cloudstateio/cloudstate/issues/536, the StatefulService entity ends up with a Status section looking like:

status:
  conditions:
  - lastTransitionTime: "2021-02-18T01:15:21Z"
    message: Stateful store no-store not supported
    reason: StoreNotSupported
    status: "True"
    type: NotReady
  - lastTransitionTime: "2021-02-18T01:25:22Z"
    message: Application is in the process of updating
    reason: UpdateInProgress
    status: "True"
    type: NotReady
marcoderama commented 3 years ago

Tests now pass but converting to draft because I want to reconsider making the Status component optional.