Adds Watches to all Services to trigger a reconcile request if a custom-resource-owned Service is ever manually updated to fix Service objects owned by this operator
Adds chia-healthcheck to the list of optional sidecar containers for seeders and full_nodes
And also:
Slimmed down controller functions that had unused arguments and receivers. Removing them makes the functions easier to test, and easier to keep track of what the function actually does.
Reorganizes some controller code to use a chia container builder function to have a consistent way to build the chia container and reduce code duplication and increase testability
Reorganize some Service assembler function logic to reduce a bit of code duplication and increase testability
Turns off zap's Development mode logger, which mostly means we won't get stack traces on warning logs (only error logs) with the production logger
Main features of this PR:
And also: