Helm chart generation now incudes crds and uses the default service account with no resource limits.

Author: trevorbox

Makefile

@@ -127,8 +127,10 @@ bundle-build:
 # Generate helm chart
 helmchart: kustomize
 	mkdir -p ./charts/${OPERATOR_NAME}/templates
+	mkdir -p ./charts/${OPERATOR_NAME}/crds
 	cp ./config/helmchart/templates/* ./charts/${OPERATOR_NAME}/templates
 	$(KUSTOMIZE) build ./config/helmchart | sed 's/release-namespace/{{.Release.Namespace}}/' > ./charts/${OPERATOR_NAME}/templates/rbac.yaml
+	$(KUSTOMIZE) build ./config/crd > ./charts/${OPERATOR_NAME}/crds/crds.yaml
 	version=${VERSION} envsubst < ./config/helmchart/Chart.yaml.tpl  > ./charts/${OPERATOR_NAME}/Chart.yaml
 	version=${VERSION} image_repo=$${IMG%:*} envsubst < ./config/helmchart/values.yaml.tpl  > ./charts/${OPERATOR_NAME}/values.yaml
 	helm lint ./charts/${OPERATOR_NAME}	

README.md

@@ -250,6 +250,29 @@ oc login --token ${token}
 make run ENABLE_WEBHOOKS=false
 ```
 
+### Test helm chart locally
+
+Define an image and tag. For example...
+
+```shell
+export imageRepository="quay.io/redhat-cop/namespace-configuration-operator"
+export imageTag="v1.0.2"
+```
+
+Deploy chart...
+
+```shell
+make helmchart IMG=${imageRepository} VERSION=${imageTag}
+helm upgrade -i namespace-configuration-operator-helmchart-test charts/namespace-configuration-operator -n namespace-configuration-operator-local --create-namespace
+```
+
+Delete...
+
+```shell
+helm delete namespace-configuration-operator-helmchart-test -n namespace-configuration-operator-local
+kubectl delete -f charts/namespace-configuration-operator/crds/crds.yaml
+```
+
 ### Building/Pushing the operator image
 
 ```shell

config/helmchart/templates/manager.yaml

@@ -22,7 +22,6 @@ spec:
       imagePullSecrets:
         {{- toYaml . | nindent 8 }}
       {{- end }}
-      serviceAccountName: {{ include "namespace-configuration-operator.serviceAccountName" . }}    
       containers:
       - command:
         - /manager

config/helmchart/values.yaml.tpl

@@ -14,21 +14,9 @@ imagePullSecrets: []
 nameOverride: ""
 fullnameOverride: ""
 
-serviceAccount:
-  # Specifies whether a service account should be created
-  create: true
-  # Annotations to add to the service account
-  annotations: {}
-  # The name of the service account to use.
-  # If not set and create is true, a name is generated using the fullname template
-  name: ""
-
 podAnnotations: {}
 
 resources:
-  limits:
-    cpu: 100m
-    memory: 30Mi
   requests:
     cpu: 100m
     memory: 20Mi