update spec when updating status

[FxKu]

Four years ago I moved the setStatus code to the k8sutil package in order to update the status also in the controller scope and not only inside cluster. For this I removed the line c.setSpec(newspec) and left it to the caller to update Spec struct.

However, it was not reflected by all callers which leads to the situation that further Update calls during DELETE events (like removing the finalizer) can be blocked because

1. the resourceVersion is already higher that it what is stored in c.Spec, in case the status changed (e.g. Running -> UpdateFailed).
2. Delete() func is not calling c.setSpec unlike the Update() and Create() because without finalizer the Postgresql resource is already gone.

Instead of implementing extra logic to Delete() this PR attempts to bring back the behavior from 4 years ago to update the spec when updating the status.

[jopadi]

👍

[jopadi]

👍

[FxKu]

👍