cr: add comments on the upstream code

Author: gkech

internal/patroni/reconcile.go

@@ -150,6 +150,8 @@ func instanceProbes(cluster *v1beta1.PostgresCluster, container *corev1.Containe
 	// TODO(cbandy): Consider if a PreStop hook is necessary.
 	container.LivenessProbe = probeTiming(cluster.Spec.Patroni)
 	container.LivenessProbe.InitialDelaySeconds = 3
+	// Create the probe handler through a constructor for the liveness probe.
+	// Introduced with K8SPG-708.
 	container.LivenessProbe.ProbeHandler = livenessProbe(cluster)
 
 	// Readiness is reflected in the controlling object's status (e.g. ReadyReplicas)
@@ -159,9 +161,15 @@ func instanceProbes(cluster *v1beta1.PostgresCluster, container *corev1.Containe
 	// of the leader Pod in the leader Service.
 	container.ReadinessProbe = probeTiming(cluster.Spec.Patroni)
 	container.ReadinessProbe.InitialDelaySeconds = 3
+	// Create the probe handler through a constructor for the readiness probe.
+	// Introduced with K8SPG-708.
 	container.ReadinessProbe.ProbeHandler = readinessProbe(cluster)
 }
 
+// livenessProbe is a custom constructor for the liveness probe.
+// This allows for more sophisticated logic to determine whether
+// the database container is considered "alive" beyond basic checks.
+// Introduced with K8SPG-708.
 func livenessProbe(cluster *v1beta1.PostgresCluster) corev1.ProbeHandler {
 	if cluster.CompareVersion("2.7.0") >= 0 {
 		return corev1.ProbeHandler{
@@ -179,6 +187,10 @@ func livenessProbe(cluster *v1beta1.PostgresCluster) corev1.ProbeHandler {
 	}
 }
 
+// readinessProbe is a custom constructor for the liveness probe.
+// This allows for more sophisticated logic to determine whether
+// the database container is considered "alive" beyond basic checks.
+// Introduced with K8SPG-708.
 func readinessProbe(cluster *v1beta1.PostgresCluster) corev1.ProbeHandler {
 	if cluster.CompareVersion("2.7.0") >= 0 {
 		return corev1.ProbeHandler{

internal/patroni/reconcile_test.go

@@ -114,6 +114,7 @@ func TestInstanceConfigMap(t *testing.T) {
 func TestInstancePod(t *testing.T) {
 	t.Parallel()
 
+	// K8SPG-708 introduced the refactoring to this unit test.
 	tests := map[string]struct {
 		expectedSpec string
 		labels       map[string]string