luarx opened a new issue, #978:
URL: https://github.com/apache/apisix-helm-chart/issues/978
Chart version: apisix-ingress-controller v1.2.0
The deployment template renders pod labels exclusively via selectorLabels:
```
template:
metadata:
labels:
{{- include "apisix-ingress-controller-manager.selectorLabels" . |
nindent 8 }}
```
There is no podLabels (or equivalent) value that gets merged into the pod
template labels, making it impossible to add custom labels to pods for
observability, scheduling, or policy purposes.
The apisix chart in the same repo supports this via
`service.labelsOverride`. A straightforward fix for the ingress controller
would be a deployment.podLabels value merged alongside
selectorLabels:
```
labels:
{{- include "apisix-ingress-controller-manager.selectorLabels" . |
nindent 8 }}
{{- with .Values.deployment.podLabels }}
{{- toYaml . | nindent 8 }}
{{- end }}
```
Note: `deployment.podAnnotations` is already supported. The missing piece
is labels only.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]