Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Helm v3 and namespaces considerations #3211

Closed
alpeb opened this issue Aug 8, 2019 · 1 comment
Closed

Helm v3 and namespaces considerations #3211

alpeb opened this issue Aug 8, 2019 · 1 comment

Comments

@alpeb
Copy link
Member

alpeb commented Aug 8, 2019

We decided not honoring Helm's --namespace option because it would make helm install create the namespace itself without the metadata we require and it would also fail when it attempts to create it again when it stumbles across our namespace.yaml template.

That was for v2. In Helm v3 --namespace doesn't attempt to create the namespace. Users are expected to use a separate tool for that, and if the namespace is not already there then helm install will fail (it doesn't even get to the point of namespace.yaml unlike above with v2).

If the user doesn't use --namespace in v3 then linkerd will get properly installed in the linkerd namespace, but the secret used by Helm to keep track of the release (now that Tiller is gone) will live in the default namespace. I'm not sure how can that affect tightly restricted clusters.

I just wanted for us to be aware of this new behavior in v3. If we wanted to support --namespace we'd have to remove namespace.yaml from the chart and provide instructions on how to generate the namespace beforehand with the appropriate metadata.

@alpeb
Copy link
Member Author

alpeb commented Aug 13, 2019

Closing for now, as there are no actionable items for Helm v2. Might revisit when v3 comes out, and after user feedback.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant