Currently, the Synapse.Status.State is simply set to RUNNING when the Reconciler managed to run the happy path once. It's the last thing done by the controller before exiting without requeing (return ctrl.Result{}, nil).
That's not very robust. First we don't check that the pods are actually running, or that Synapse is actually up. Also, subsequent reconciliations (through updates of the CR for instance) might not run successfully. The Synapse.Status.State still would show RUNNING.
Currently, the
Synapse.Status.Stateis simply set toRUNNINGwhen the Reconciler managed to run the happy path once. It's the last thing done by the controller before exiting without requeing (return ctrl.Result{}, nil).That's not very robust. First we don't check that the pods are actually running, or that Synapse is actually up. Also, subsequent reconciliations (through updates of the CR for instance) might not run successfully. The
Synapse.Status.Statestill would showRUNNING.