No pre-delete hook in zookeeper-operator chart

[jkhalack]

Description

The zookeeper-operator helm chart does not have a pre-delete hook in it.
As a result, the operator can be uninstalled with orphan custom resources hanging around.

The orphan custom resources cannot be deleted, because they have finalizers that are supposed to be cleared by the operator. However, the operator is gone at this point.

Even the operator is reinstalled, it most probably won't be able to clean up the above custom resource if its parent namespace had been marked for deletion and is stuck in the terminated state because of that orphan custom resource.

Importance

should-have

Location

zookeeper-operator helm chart

Suggestions for an improvement

Please, add a pre-delete hook to the operator chart that would verify that all custom resources have been cleaned up before deleting the operator.