[Bug/Feature]: Configmap is overriding icinga2 scripts folder

[MTSym]

Affected Chart

icinga2

Which version of the app contains the bug?

0.1.0

Please describe your problem

I wanted to use mail notifications with the director and an msmtp configuration but the icinga2 configmaps are overriding the /etc/icinga2 directory containing the scripts to send those mail notifications.

helm-charts/charts/icinga-stack/charts/icinga2/templates/statefulset.yaml

Line 52 in 11914fc

- name: {{ include "icinga2.fullname" . }}-config

It would be great if those scripts could be preserved while overriding the mount path.

I marked the Issue as Bug/Feature because with patching from kustomize it would be already possible to specify the msmtp configuration, but without those scripts you are unable to use it.

It would also be great if the msmtp configuration is exposed to the values.yml

[mocdaniel]

Thank you for this issue and sorry for the longer-than-usual reaction time. We will have to revisit our thoughts regarding what we mount where for the Icinga2 workload(s), but I agree that we should preserve those scripts provided by the Icinga project.

Regarding your request to include the msmtp configuration in the values.yml, this would need community input. As of now, none of the maintainers could implement such integration due to lack of experience with msmtp.