WIP: don't install CRDs of disabled subcharts

[MBrouns]

Possible solution for #938

[stefanprodan]

If this actually works, we'll need it on upgrade too.

[MBrouns]

That makes sense. I could probably add it to applyCRDs instead to cover all the use-cases, but that would mean changing the signature of that function to also include the vals. Is that better?

[souleb]

This results in helmchartutil.ProcessDependenciesWithMerge being called twice. We need a test case for idempotence here.

[MBrouns]

I can put in some time tomorrow or Thursday I think to write test cases and do it on upgrade as well. Any other concerns that need to be addressed?

[stefanprodan]

We need a test case for idempotence here.

Reading this I hope we are safe:

• https://github.com/helm/helm/blob/main/pkg/chartutil/dependencies.go#L208-L211
• https://github.com/helm/helm/blob/main/pkg/chartutil/dependencies.go#L303-L315

In any case, if anywhere in that logic a map from the chart is touched, calling the function twice will mess up the whole thing, so we really need a test.