Uninstall doesn't check namespace

[mattreyuk]

I was working on a group of replication controllers and services that I created with helm in the default namespace. Later I created another version exactly the same in a different namespace (namespace created outside of helm). I ran helm uninstall on the version of the chart with a namespace specified in the manifest files and helm deleted the controllers and services in the default namespace.

helm version 0.3.1+d4c0fa8

[mboersma]

+1. I assume this is a bug. I've been working around it with helm uninstall -n <namespace>.

[technosophos]

Has anyone tested this behavior with a raw manifest file and kubectl? For deletes, we drop to kubectl. I'm going to look at the code and see what the easiest path to a fix is. Thanks, @mattreyuk, for the report.

[technosophos]

We believe we've verified that kubectl ignores namespaces in the delete case. Our stop-gap solution is to require -n NAMESPACE when running an uninstall command.

Longer term, we need to decide how to handle this case. There are a couple of interesting edge cases where charts have multiple namespaces. We need to figure out how to solve those.