add test

Author: egegunes

internal/controller/postgrescluster/controller_test.go

@@ -512,6 +512,31 @@ spec:
 			Expect(icm.Data["patroni.yaml"]).ToNot(BeZero())
 		})
 
+		It("doesn't reconcile ConfigMap with override annotation", func() {
+			icm := &corev1.ConfigMap{}
+			Expect(suite.Client.Get(context.Background(), client.ObjectKey{
+				Namespace: test.Namespace.Name, Name: instance.Name + "-config",
+			}, icm)).To(Succeed())
+
+			Expect(icm.Labels[naming.LabelCluster]).To(Equal("carlos"))
+			Expect(icm.Labels[naming.LabelInstance]).To(Equal(instance.Name))
+			Expect(icm.Data["patroni.yaml"]).ToNot(BeZero())
+
+			icm.SetAnnotations(map[string]string{
+				naming.OverrideConfigAnnotation: "true",
+			})
+			icm.Data["patroni.yaml"] = ""
+
+			Expect(suite.Client.Update(context.Background(), icm)).To(Succeed())
+
+			Expect(suite.Client.Get(context.Background(), client.ObjectKey{
+				Namespace: test.Namespace.Name, Name: instance.Name + "-config",
+			}, icm)).To(Succeed())
+
+			Expect(icm.Annotations[naming.OverrideConfigAnnotation]).To(Equal("true"))
+			Expect(icm.Data["patroni.yaml"]).To(BeZero())
+		})
+
 		Specify("Instance StatefulSet", func() {
 			Expect(instance.Labels[naming.LabelCluster]).To(Equal("carlos"))
 			Expect(instance.Labels[naming.LabelInstanceSet]).To(Equal("samba"))

internal/naming/annotations.go

@@ -70,6 +70,7 @@ const (
 	// touch cloud-based backups.
 	AuthorizeBackupRemovalAnnotation = annotationPrefix + "authorizeBackupRemoval"
 
+	// K8SPG-712
 	// OverrideConfigAnnotation is an annotation used to prevent the controller from reconciling
 	// ConfigMaps when a user wants to manually override their contents. When this annotation
 	// is present, the controller will not update the ConfigMap, allowing users to make custom