[bitnami/kafka] add missed service annotations for external access service

[xud6]

Description of the change

add missed service annotations for external access service of bitnami/kafka chart

Benefits

allow add annotations to external access service for configure of loadbalancer and etc

Possible drawbacks

Applicable issues

Additional information

Checklist

• Chart version bumped in Chart.yaml according to semver.
• Variables are documented in the README.md
• Title of the PR starts with chart name (e.g. [bitnami/chart])
• If the chart contains a values-production.yaml apart from values.yaml, ensure that you implement the changes in both files

[carrodher]

Thanks for your contribution!

Changes LGTM (I just removed the $root.Values to unify the annotations with the rest of files)

[xud6]

@carrodher
It can't render without '$root.Value' on my machine

helm install ./bitnami/kafka/ --dry-run --debug --set externalAccess.enabled=true

Error: render error in "kafka/templates/svc-external-access.yaml": template: kafka/templates/svc-external-access.yaml:17:65: executing "kafka/templates/svc-external-access.yaml" at <.Values.externalAcce...>: can't evaluate field Values in type int

[carrodher]

Yes, you're right. I did this change to unify all the templates:

$ ag 'annotations: ' templates

templates/svc.yaml
7:  annotations: {{ include "kafka.tplValue" ( dict "value" .Values.service.annotations "context" $) | nindent 4 }}

templates/kafka-jmx-metrics-svc.yaml
8:  annotations: {{ include "kafka.tplValue" ( dict "value" .Values.metrics.jmx.service.annotations "context" $) | nindent 4 }}

templates/kafka-metrics-svc.yaml
8:  annotations: {{ include "kafka.tplValue" ( dict "value" .Values.metrics.kafka.service.annotations "context" $) | nindent 4 }}

But in this case, it is not possible as we are in a range.
Thanks for report it, I just fixed it here: 94f836f