Helm-operator: Operator does not support `--skip-crds`

[sayboras]

Describe the bug
Failed to install helm charts with crds, the same can be done through helm cli with ---skip-crds

To Reproduce
Trying to configure helm release as per below config:

apiVersion: helm.fluxcd.io/v1
kind: HelmRelease
metadata:
  name: kong-dev
  namespace: dev
  annotations:
    fluxcd.io/automated: "true"
spec:
  releaseName: kong-dev
  chart:
    repository: https://charts.konghq.com/
    name: kong
    version: 1.1.1

Expected behavior
HelmRelease should be created successfully

Logs

ts=2020-02-10T11:57:15.875705093Z caller=release.go:214 component=release release=kong-dev targetNamespace=dev resource=dev:helmrelease/kong-dev helmVersion=v3 error="Helm release failed" revision=1.1.1 err="failed to upgrade chart for release [kong-dev]: rendered manifests contain a resource that already exists. Unable to continue with install: existing resource conflict: kind: CustomResourceDefinition, namespace: , name: kongconsumers.configuration.konghq.com"

Additional context
Add any other context about the problem here, e.g

• Helm operator version: helm-operator-0.6.0
• Targeted Helm version:

version.BuildInfo{Version:"v3.0.2", GitCommit:"19e47ee3283ae98139d98460de796c1be1e3975f", GitTreeState:"clean", GoVersion:"go1.13.5"}

• Kubernetes version:

Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:30:10Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.9-eks-c0eccc", GitCommit:"c0eccca51d7500bb03b2f163dd8d534ffeb2f7a2", GitTreeState:"clean", BuildDate:"2019-12-22T23:14:11Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}

[stefanprodan]

There is a installCRDs boolean in that chart, set it to false and add the CRDs to your Flux git repo.

[hiddeco]

@stefanprodan while this is true for this chart, I still think it would be good to integrate the flag into the operator for charts that for some reason do not make it possible to disable the creation of CRDs through their values.

[stefansedich]

Looks like the kong chart also states this so really need the --skip-crds for installs.

Create CRDs. Regardless of value of this, Helm v3+ will install the CRDs if those are not present already. Use `--skip-crds` with `helm install` if you want to skip CRD creation.

@hiddeco assign this to me will pick it up tonight.

[sayboras]

Looks like the kong chart also states this so really need the --skip-crds for installs.

Create CRDs. Regardless of value of this, Helm v3+ will install the CRDs if those are not present already. Use `--skip-crds` with `helm install` if you want to skip CRD creation.

@hiddeco yes, I tried before with CLI, and it requires passing the flag. Thanks for picking up this one 💯