Apparmor annotation still not taking affect about helmchart

[cakrit]

@josecv and @charlieegan3 any comments on this? Should we do another PR to make the changes above?

[josecv]

@cakrit yup, with some minor adjustments -- basically the with .Values.slave.podAnnotations needs to be moved to be under checksum/config so that all annotations get merged together:

        container.apparmor.security.beta.kubernetes.io/netdata: unconfined
        checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
 {{- with .Values.slave.podAnnotations }}
 {{ toYaml . | trim | indent 8 }}
 {{- end }}

I'd also recommend having a way to turn off the appArmor annotation btw

[cakrit]

Feel free to make a PR gents. I'll be pretty swamped for the next week or two. Will try it if no one else has a go in the meantime.

[charlieegan3]

I've had a go at implementing this here: #41