[GitHub] [pulsar-helm-chart] jeantil commented on issue #66: Helm charts are managing namespace

[GitBox <gi...@apache.org>]

jeantil commented on issue #66:
URL: https://github.com/apache/pulsar-helm-chart/issues/66#issuecomment-723193163


   I started an attempt at implementing helm's 3 best practices regarding namespace management as laid out by one helm's core maintainer in [this comment](https://github.com/helm/helm/issues/5465#issuecomment-473942223)
   
   However after trying to apply the namespace-less recommendation I encoutered at  [2 issues](https://github.com/helm/helm/issues/5465#issuecomment-723182962) which made me doubt that the official best practice was sound 
   
   I am considering using defaulting instead of fully removing the namespace mentions 
   ```
   {{ .Values.namespace | default .Release.Namespace }}
   ```
   This would make the default behaviour align with the recommendations while allowing users to pin the namespace through value
   I wonder if people in the team have an opinion ...


----------------------------------------------------------------
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.

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