[I] feat(apisix): Add namespace to manifest templates [apisix-helm-chart]

["vramperez (via GitHub)" <gi...@apache.org>]

vramperez opened a new issue, #949:
URL: https://github.com/apache/apisix-helm-chart/issues/949

   ## Context
   
   Some manifest templates in the APISIX chart do not include `namespace: {{ .Release.Namespace }}` 
   in their metadata, for example:
   - [templates/apisix-config-cm.yml](https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix/templates/apisix-config-cm.yml)
   - [templates/ingress.yml](https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix/templates/ingress.yaml)
   
   This is inconsistent with Helm best practices, where templates should respect 
   the namespace provided during installation via `--namespace` or `.Release.Namespace`.
   
   ## Expected behavior
   
   All manifest templates should include:
   
       metadata:
         namespace: {{ .Release.Namespace }}
   
   
   Is this an intentional design decision?
   
   If not, would a PR adding `namespace: {{ .Release.Namespace }}` to the affected 
   templates be welcome?


-- 
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: notifications-unsubscribe@apisix.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org