feat: add garbage collection (#38)

Makefile

@@ -104,8 +104,18 @@ kind-test: docker-build ## Deploy including test
 	kustomize build config/base/crd | kubectl --context kind-${CLUSTER} apply -f -	
 	kind load docker-image ${IMG} --name ${CLUSTER}
 	kubectl -n k8sgrowthbook-system delete pods --all
-	kustomize build config/tests/cases/${TEST_PROFILE} --enable-helm | kubectl --context kind-${CLUSTER} apply -f -	
+
+	echo "pre-test"
+	kustomize build config/tests/cases/${TEST_PROFILE}/pre-test --enable-helm | kubectl --context kind-${CLUSTER} apply -f -	
 	kubectl --context kind-${CLUSTER} -n k8sgrowthbook-system wait --for=condition=Ready pods -l control-plane=controller-manager -l app.kubernetes.io/managed-by!=Helm -l verify!=yes --timeout=3m
+
+	echo "test"
+	kustomize build config/tests/cases/${TEST_PROFILE}/test --enable-helm | kubectl --context kind-${CLUSTER} apply -f -	
+	kubectl --context kind-${CLUSTER} -n k8sgrowthbook-system wait --for=jsonpath='{.status.conditions[1].reason}'=PodCompleted pods -l control-plane=controller-manager -l app.kubernetes.io/managed-by!=Helm -l verify=yes --timeout=3m
+
+	echo "post-test"
+	kustomize build config/tests/cases/${TEST_PROFILE}/test --enable-helm | kubectl --context kind-${CLUSTER} delete -f -	
+	kustomize build config/tests/cases/${TEST_PROFILE}/post-test --enable-helm | kubectl --context kind-${CLUSTER} apply -f -	
 	kubectl --context kind-${CLUSTER} -n k8sgrowthbook-system wait --for=jsonpath='{.status.conditions[1].reason}'=PodCompleted pods -l control-plane=controller-manager -l app.kubernetes.io/managed-by!=Helm -l verify=yes --timeout=3m
 
 ##@ Deployment

api/v1beta1/growthbookinstance_types.go

@@ -28,6 +28,10 @@ type GrowthbookInstanceSpec struct {
 	// Interval reconciliation
 	Interval *metav1.Duration `json:"interval,omitempty"`
 
+	// Prune
+	// +kubebuilder:validation:Required
+	Prune bool `json:"prune"`
+
 	// Timeout while reconciling the instance
 	// +kubebuilder:default:="5m"
 	Timeout *metav1.Duration `json:"timeout,omitempty"`

api/v1beta1/meta.go

@@ -26,6 +26,7 @@ const (
 	SynchronizedReason = "Synchronized"
 	ProgressingReason  = "Progressing"
 	FailedReason       = "Failed"
+	Finalizer          = "finalizers.doodle.com"
 )
 
 // ConditionalResource is a resource with conditions

chart/k8sgrowthbook-controller/Chart.yaml

@@ -15,4 +15,4 @@ keywords:
 name: k8sgrowthbook-controller
 sources:
 - https://github.com/DoodleScheduling/k8sgrowthbook-controller
-version: 0.1.0
+version: 0.1.1

chart/k8sgrowthbook-controller/crds/growthbook.infra.doodle.com_growthbookinstances.yaml

@@ -71,6 +71,9 @@ spec:
                     description: Address is a MongoDB comptaible URI `mongodb://xxx`
                     type: string
                 type: object
+              prune:
+                description: Prune
+                type: boolean
               resourceSelector:
                 description: ResourceSelector defines a selector to select Growthbook
                   resources associated with this instance
@@ -124,6 +127,8 @@ spec:
                 default: 5m
                 description: Timeout while reconciling the instance
                 type: string
+            required:
+            - prune
             type: object
           status:
             description: GrowthbookInstanceStatus defines the observed state of GrowthbookInstance

config/base/crd/bases/growthbook.infra.doodle.com_growthbookinstances.yaml

@@ -71,6 +71,9 @@ spec:
                     description: Address is a MongoDB comptaible URI `mongodb://xxx`
                     type: string
                 type: object
+              prune:
+                description: Prune
+                type: boolean
               resourceSelector:
                 description: ResourceSelector defines a selector to select Growthbook
                   resources associated with this instance
@@ -124,6 +127,8 @@ spec:
                 default: 5m
                 description: Timeout while reconciling the instance
                 type: string
+            required:
+            - prune
             type: object
           status:
             description: GrowthbookInstanceStatus defines the observed state of GrowthbookInstance

config/tests/base/catalog.yaml → config/tests/base/catalog/catalog.yaml

@@ -5,6 +5,7 @@ metadata:
 spec:
   interval: 5m
   suspend: false
+  prune: false
   mongodb:
     uri: mongodb://mongodb.k8sgrowthbook-system:27017/growthbook
     rootSecret:

config/tests/base/catalog/kustomization.yaml

@@ -0,0 +1,5 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+
+resources:
+- catalog.yaml

config/tests/base/verify-get-features-clienttoken/kustomization.yaml

@@ -0,0 +1,5 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+
+resources:
+- verify-get-features-clienttoken.yaml

config/tests/cases/growthbook-v2.1.1-mongodb-v6/kustomization.yaml → config/tests/cases/growthbook-v2.1.1-mongodb-v5/post-test/kustomization.yaml

@@ -3,8 +3,4 @@ kind: Kustomization
 namespace: k8sgrowthbook-system
 
 resources:
-- ../../base
-
-images:
-- name: growthbook/growthbook
-  newTag: 2.1.1
+- ../../../base/verify-get-features-clienttoken

config/tests/cases/growthbook-v2.1.1-mongodb-v5/kustomization.yaml → config/tests/cases/growthbook-v2.1.1-mongodb-v5/pre-test/kustomization.yaml

@@ -3,7 +3,9 @@ kind: Kustomization
 namespace: k8sgrowthbook-system
 
 resources:
-- ../../base
+- ../../../base/mongodb
+- ../../../base/growthbook/backend
+- ../../../base/growthbook-controller
 
 images:
 - name: growthbook/growthbook

config/tests/cases/growthbook-v2.1.1-mongodb-v5/test/kustomization.yaml

@@ -0,0 +1,7 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/catalog
+- ../../../base/verify-get-features-clienttoken

config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-proxy/post-test/kustomization.yaml

@@ -0,0 +1,6 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/verify-get-features-clienttoken

config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-proxy/kustomization.yaml → config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-proxy/pre-test/kustomization.yaml

@@ -3,9 +3,10 @@ kind: Kustomization
 namespace: k8sgrowthbook-system
 
 resources:
-- ../../base
-- ../../base/growthbook/proxy
-- verify-proxy-get-features-clienttoken.yaml
+- ../../../base/mongodb
+- ../../../base/growthbook/backend
+- ../../../base/growthbook/proxy
+- ../../../base/growthbook-controller
 
 images:
 - name: growthbook/growthbook
@@ -26,4 +27,4 @@ patches:
       value: "http://growthbook-proxy"
     - op: replace
       path: /data/PROXY_HOST_PUBLIC
-      value: "http://growthbook-proxy"
+      value: "http://growthbook-proxy"

config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-proxy/test/kustomization.yaml

@@ -0,0 +1,7 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/catalog
+- verify-proxy-get-features-clienttoken.yaml

config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-prune/post-test/kustomization.yaml

@@ -0,0 +1,6 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- verify-get-features-clienttoken.yaml

config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-prune/post-test/verify-get-features-clienttoken.yaml

@@ -0,0 +1,31 @@
+apiVersion: v1
+kind: Pod
+metadata:
+  name: verify-get-features-clienttoken
+  labels:
+    verify: yes
+spec:
+  restartPolicy: OnFailure
+  containers:
+  - image: curlimages/curl:8.1.2
+    imagePullPolicy: IfNotPresent
+    name: verify
+    command:
+    - /bin/sh
+    - "-c"
+    - |
+      code=$(curl -s -o /dev/null -I -w "%{http_code}" http://growthbook-api/api/features/sdk-token)
+      if [ "$code" = "400" ]; then
+        exit 0
+      else
+        exit 1
+      fi
+    resources: {}
+    securityContext:
+      allowPrivilegeEscalation: false
+      readOnlyRootFilesystem: false
+      runAsGroup: 1000
+      runAsNonRoot: true
+      runAsUser: 1000
+    terminationMessagePath: /dev/termination-log
+    terminationMessagePolicy: File

config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-prune/pre-test/kustomization.yaml

@@ -0,0 +1,12 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/mongodb
+- ../../../base/growthbook/backend
+- ../../../base/growthbook-controller
+
+images:
+- name: growthbook/growthbook
+  newTag: 2.1.1

config/tests/cases/growthbook-v2.1.1-mongodb-v6-with-prune/test/kustomization.yaml

@@ -0,0 +1,16 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/catalog
+- ../../../base/verify-get-features-clienttoken
+
+patches:
+- target:
+    kind: GrowthbookInstance
+    name: my-instance
+  patch: |-
+    - op: replace
+      path: /spec/prune
+      value: true

config/tests/cases/growthbook-v2.1.1-mongodb-v6/post-test/kustomization.yaml

@@ -0,0 +1,6 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/verify-get-features-clienttoken

config/tests/cases/growthbook-v2.1.1-mongodb-v6/pre-test/kustomization.yaml

@@ -0,0 +1,12 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/mongodb
+- ../../../base/growthbook/backend
+- ../../../base/growthbook-controller
+
+images:
+- name: growthbook/growthbook
+  newTag: 2.1.1

config/tests/cases/growthbook-v2.1.1-mongodb-v6/test/kustomization.yaml

@@ -0,0 +1,7 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+namespace: k8sgrowthbook-system
+
+resources:
+- ../../../base/catalog
+- ../../../base/verify-get-features-clienttoken