Helm v3 changes

[almahmoud]

Migrations still need to be done, but I thought I'd PR code changes first for collective input

[coveralls]

Coverage increased (+1.4%) to 90.292% when pulling f517524 on helm_v3 into 7195774 on master.

[almahmoud]

Tests finally passed. Let me know if I should change anything else

[nuwang]

Thanks for these changes. I think we are pretty close. Now that Helmsman wraps both Helm and some Kubernetes functionality, the name doesn't seem as clean anymore, but I guess that's ok. I imagine in future we'll need an OperaMan as well :-)

[nuwang]

After this change, I'm guessing we'll no longer be able to use the default namespace, because projman will raise an error if you try to create a project with an existing namespace. I think this is a good thing, because default is an implicit namespace, and we probably want to avoid exposing things that get installed to default for security reasons.
@almahmoud @afgane Any thoughts on what the new default namespace could be called? "initial" or "starter" perhaps?

[almahmoud]

I don't mind changing the initial project, but I did special-case default to not throw an error even if namespace exists and not delete it (https://github.com/galaxyproject/cloudman/pull/105/files#diff-3d0ef3ae18103076ef9de75ea24376bdR105). Since kubectl will install into default namespace when no namespace is specified, I thought it would be good to have the project by default. I would even say let's make the project non-deletable as well potentially?

[nuwang]

I think we should have a new class called MockKube or something? These two things don't belong together I think.

[almahmoud]

But we are ultimately just replacing the run_command. I guess we could separate them into separate parsers, but in the end, for the mock client the command that matters is mock_run_command
everything else is just helpers, so I would argue that the client should be a single mock client. I separated the parsers for helm and kubectl commands though in 88ab6e6

[almahmoud]

All changes now working, and made some more bug fixes as it was working with mock but not the real helm client (helm 3 does not list releases from other namespaces by default).
Also packaged and working in the wild on am-dev, with both cloudman-boot and cloudman helm v3

[almahmoud]

I believe it's now ready to merged, along with CloudVE/cloudman-helm#12 and CloudVE/cloudman-boot#10. Can be tested as am-dev Cloudman 2.0 appliance on CL.