The operator no longer installs the CRDs on start-up (#726)

Author: thegridman

.github/workflows/build.yaml

@@ -140,11 +140,16 @@ jobs:
 
     - name: E2E Remote Tests
       shell: bash
-      run: make e2e-test
+      run: |
+        make undeploy
+        make uninstall-crds
+        make e2e-test
 
     - name: Helm Chart Tests
       shell: bash
       run: |
+        make undeploy
+        make uninstall-crds
         make e2e-helm-test
 
     - name: Upload Manifests

.github/workflows/istio-tests.yaml

@@ -105,7 +105,6 @@ jobs:
         sudo echo nameserver 8.8.8.8 > /run/systemd/resolve/stub-resolv.conf
 
     - name: Start KinD Cluster
-#     Start a KinD K8s cluster with single worker node
       shell: bash
       run: |
         echo "${{ secrets.GITHUB_TOKEN }}" | docker login ghcr.io -u $ --password-stdin
@@ -119,7 +118,6 @@ jobs:
         make all
 
     - name: Load Images to KinD
-#     Load the images just built to the KinD cluster
       shell: bash
       run: |
         make kind-load
@@ -130,8 +128,9 @@ jobs:
     - name: Istio Tests
       shell: bash
       run: |
-        make deploy
+        make reset-namespace
         ISTIO_VERSION=${{ matrix.istioVersion }} make install-istio
+        make deploy
         make e2e-client-test
         make e2e-test
         make undeploy

Makefile

@@ -732,13 +732,25 @@ $(BUILD_TARGETS)/java: $(JAVA_FILES)
 .PHONY: helm-chart
 helm-chart: $(BUILD_PROPS) $(BUILD_HELM)/coherence-operator-$(VERSION).tgz   ## Build the Coherence Operator Helm chart
 
+CRD_TEMPLATE := $(BUILD_HELM)/coherence-operator/templates/crd.yaml
 $(BUILD_HELM)/coherence-operator-$(VERSION).tgz: $(BUILD_PROPS) $(HELM_FILES) $(BUILD_TARGETS)/generate $(BUILD_TARGETS)/manifests $(TOOLS_BIN)/kustomize
 # Copy the Helm chart from the source location to the distribution folder
-	-mkdir -p $(BUILD_HELM)
+	-mkdir -p $(BUILD_HELM)/temp
 	cp -R ./helm-charts/coherence-operator $(BUILD_HELM)
+	$(KUSTOMIZE) build $(BUILD_DEPLOY)/overlays/helm -o $(BUILD_HELM)/temp
+	rm $(CRD_TEMPLATE) || true
+	echo "{{- if (eq .Values.installCrd true) }}" > $(CRD_TEMPLATE)
+	cat  $(BUILD_HELM)/temp/apiextensions.k8s.io_v1_customresourcedefinition_coherence.coherence.oracle.com.yaml >> $(CRD_TEMPLATE)
+	printf "\n{{- if (eq .Values.allowCoherenceJobs true) }}\n" >> $(CRD_TEMPLATE)
+	echo "---" >> $(CRD_TEMPLATE)
+	cat  $(BUILD_HELM)/temp/apiextensions.k8s.io_v1_customresourcedefinition_coherencejob.coherence.oracle.com.yaml >> $(CRD_TEMPLATE)
+	echo "" >> $(CRD_TEMPLATE)
+	echo "{{- end }}" >> $(CRD_TEMPLATE)
+	echo "{{- end }}" >> $(CRD_TEMPLATE)
 	$(call replaceprop,$(BUILD_HELM)/coherence-operator/Chart.yaml $(BUILD_HELM)/coherence-operator/values.yaml $(BUILD_HELM)/coherence-operator/templates/deployment.yaml $(BUILD_HELM)/coherence-operator/templates/rbac.yaml)
 	helm lint $(BUILD_HELM)/coherence-operator
 	helm package $(BUILD_HELM)/coherence-operator --destination $(BUILD_HELM)
+	rm -rf $(BUILD_HELM)/temp
 
 # ---------------------------------------------------------------------------
 # Do a search and replace of properties in selected files in the Helm charts.

api/v1/coherence_types.go

@@ -880,6 +880,11 @@ type CoherenceUtilsSpec struct {
 	// More info: https://kubernetes.io/docs/concepts/containers/images#updating-images
 	// +optional
 	ImagePullPolicy *corev1.PullPolicy `json:"imagePullPolicy,omitempty"`
+	// Image is used to set the utils image used in Coherence Pods.
+	//
+	// Deprecated: This field is deprecated and no longer used, any value set will be ignored.
+	// +optional
+	Image *string `json:"image,omitempty"`
 }
 
 // EnsureImage ensures that the image value is set.

api/v1/coherenceresource_utils.go

@@ -0,0 +1,9 @@
+apiVersion: kustomize.config.k8s.io/v1alpha1
+kind: Component
+
+labels:
+  - pairs:
+      control-plane: coherence
+      app.kubernetes.io/name: coherence-operator
+      app.kubernetes.io/version: "3.5.0"
+      app.kubernetes.io/part-of: coherence-operator

config/components/helm/kustomization.yaml

@@ -1,4 +1,4 @@
 - op: add
   path: /spec/template/spec/containers/0/args/-
   value:
-    - --install-job-crd=false
+    - --enable-jobs=false

config/components/no-coherencejob/no-jobs-patch.yaml

@@ -12,9 +12,6 @@
 - op: add
   path: /spec/template/spec/containers/0/args/-
   value: --enable-webhook=false
-- op: add
-  path: /spec/template/spec/containers/0/args/-
-  value: --install-crd=false
 - op: add
   path: /spec/template/spec/containers/0/args/-
   value: --node-lookup-enabled=false

config/components/restricted/single-namespace-patch.yaml

@@ -61,7 +61,6 @@ spec:
         args:
           - operator
           - --enable-leader-election
-          - --install-crd=false
         envFrom:
           - configMapRef:
               name: env-vars

config/manager/manager.yaml

@@ -0,0 +1,8 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+
+resources:
+  - ../../crd-small
+
+components:
+  - ../../components/helm