I kept the namespace value in values.yaml for backward compatibility reasons. But it would probably be better to set as
namespace: ""
Then, per default, the Releases.Namespace value is taken and you still can overwrite it in values.yaml.
Currently, if someone wants to install the helm chart, it gets installed in kube-system namespace. Even if you specify -n other-namespace . You have to actively set namespace: "" in values.yaml to make it installable in a different namespace via helm flags.
@djboris9 let me know what you think.
Edit:
Best practices is discussed here. I get the feeling that namespace: {{ .Releases.Namespace }} would be best.
Namespace should be configured via
-n
helm flag.I kept the
namespace
value invalues.yaml
for backward compatibility reasons. But it would probably be better to set asThen, per default, the
Releases.Namespace
value is taken and you still can overwrite it invalues.yaml
.Currently, if someone wants to install the helm chart, it gets installed in
kube-system
namespace. Even if you specify-n other-namespace
. You have to actively setnamespace: ""
invalues.yaml
to make it installable in a different namespace via helm flags.@djboris9 let me know what you think.
Edit:
Best practices is discussed here. I get the feeling that
namespace: {{ .Releases.Namespace }}
would be best.