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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
