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]

Reply via email to