fix(helm chart): if .Values.server.mp.enabled is set to false, honour it

[iainsproat]

Description & motivation

If .Values.server.monitoring.mp.enabled was set to false (without quotations) then ENABLE_MP was never set due to bug in helm chart logic. This PR makes it more robust, and will force the value to be quoted.

Changes:

• quotes .Values.esrver.monitoring.mp.enabled value, so false is honoured
• removes the defaulting of ENABLE_MP to true if no value is provided, as the server itself controls the default value and this is duplicate.
• disables ENABLE_MP for Docker Compose deployed by Tilt.

To-do before merge:

Screenshots:

Validation of changes:

Checklist:

• My pull request follows the guidelines in the Contributing guide?
• My pull request does not duplicate any other open Pull Requests for the same update/change?
• My commits are related to the pull request and do not amend unrelated code or documentation.
• My code follows a similar style to existing code.
• I have added appropriate tests.
• I have updated or added relevant documentation.

References