Preserve Helm Parameters case about jkube HOT 3 CLOSED

@rohanKanojia yes, I was planning on taking this on as well.

from jkube.

@bryopsida : I see you've got a PR related to helm open. Do you plan to work on this as well? Shall I assign it to you?

from jkube.

I can't really find a reason as to why we're transforming the parameter names to lower-case.

The only reason I can think of is to make them case-insensitive.

The change was added as part of #116 when the Helm logic was extracted from the Mojo to a separate module in JKube Kit.

Fabric8 Maven Plugin's original code doesn't include any case transformation either:

https://github.com/fabric8io/fabric8-maven-plugin/blob/89af88323fa685d8b298ba4b2cbc415b9b2911f0/plugin/src/main/java/io/fabric8/maven/plugin/mojo/build/HelmMojo.java#L682-L684

In conclusion, I think it's safe to preserve the case-sensitivity and I don't really see the need for adding an additional flag. I'll refactor #2369 accordingly.

from jkube.