ci: remove PostgreSQL 10 from supported versions (cloudnative-pg#1009)

Starting with next minor version 1.19, CloudNativePG will stop supporting
PostgreSQL 10, which reached EOL in November 11 2022.


Signed-off-by: Jitendra Wadle <jitendra.wadle@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>

Author: wadlejitendra

.github/pg_versions.json

@@ -18,9 +18,5 @@
   "11": [
     "11.18",
     "11.17"
-  ],
-  "10": [
-    "10.23",
-    "10.22"
   ]
 }

.github/postgres-versions-update.py

@@ -21,7 +21,7 @@
 from packaging import version
 from subprocess import check_output
 
-min_supported_major = 10
+min_supported_major = 11
 
 pg_repo_name = "cloudnative-pg/postgresql"
 pg_version_re = re.compile(r"^(\d+)(?:\.\d+|beta\d+|rc\d+|alpha\d+)(-\d+)?$")

docs/src/database_import.md

@@ -17,8 +17,8 @@ As a result, the instructions in this section are suitable for both:
 - importing one or more databases from an existing PostgreSQL instance, even
   outside Kubernetes
 - importing the database from any PostgreSQL version to one that is either the
-  same or newer, enabling *major upgrades* of PostgreSQL (e.g. from version 10.x
-  to version 14.x)
+  same or newer, enabling *major upgrades* of PostgreSQL (e.g. from version 11.x
+  to version 15.x)
 
 !!! Warning
     When performing major upgrades of PostgreSQL you are responsible for making

docs/src/e2e.md

@@ -19,7 +19,6 @@ The following PostgreSQL versions are tested:
 * PostgreSQL 13
 * PostgreSQL 12
 * PostgreSQL 11
-* PostgreSQL 10
 
 For each tested version of Kubernetes and PostgreSQL, a Kubernetes
 cluster is created using [kind](https://kind.sigs.k8s.io/),

docs/src/monitoring.md

@@ -471,7 +471,7 @@ Here is a short description of all the available fields:
     - `primary`: whether to run the query only on the primary instance <!-- wokeignore:rule=master -->
     - `master`: same as `primary` (for compatibility with the Prometheus PostgreSQL exporter's syntax - deprecated) <!-- wokeignore:rule=master -->
     - `runonserver`: a semantic version range to limit the versions of PostgreSQL the query should run on
-       (e.g. `">=10.0.0"` or `">=12.0.0 <=15.0.0"`)
+       (e.g. `">=11.0.0"` or `">=12.0.0 <=15.0.0"`)
     - `target_databases`: a list of databases to run the `query` against,
       or a [shell-like pattern](#example-of-a-user-defined-metric-running-on-multiple-databases)
       to enable auto discovery. Overwrites the default database if provided.

docs/src/replication.md

@@ -59,9 +59,6 @@ CREATE USER streaming_replica WITH REPLICATION;
    -- NOSUPERUSER INHERIT NOCREATEROLE NOCREATEDB NOBYPASSRLS
 ```
 
-!!! Note
-    Due to a `pg_rewind` requirement, in PostgreSQL 10 the `streaming_replica`
-    user is created with `SUPERUSER` privileges.
 
 Out of the box, the operator automatically sets up streaming replication within
 the cluster over an encrypted channel and enforces TLS client certificate

internal/management/controller/instance_controller.go

@@ -238,15 +238,10 @@ func (r *InstanceReconciler) Reconcile(
 }
 
 func (r *InstanceReconciler) configureSlotReplicator(cluster *apiv1.Cluster) {
-	// If PostgreSQL is older than 11 never start the SlotReplicator
-	psqlVersion, err := cluster.GetPostgresqlVersion()
-	if err != nil || psqlVersion < 110000 {
+	switch r.instance.PodName {
+	case cluster.Status.CurrentPrimary, cluster.Status.TargetPrimary:
 		r.instance.ConfigureSlotReplicator(nil)
-	}
-
-	if cluster.Status.CurrentPrimary == r.instance.PodName || cluster.Status.TargetPrimary == r.instance.PodName {
-		r.instance.ConfigureSlotReplicator(nil)
-	} else {
+	default:
 		r.instance.ConfigureSlotReplicator(cluster.Spec.ReplicationSlots)
 	}
 }

tests/e2e/asserts_test.go

@@ -2480,12 +2480,6 @@ func AssertReplicationSlotsOnPod(
 	clusterName string,
 	pod corev1.Pod,
 ) {
-	// Replication slot high availability requires PostgreSQL 11 or above
-	if env.PostgresVersion == 10 {
-		Skip("Ignoring replication slots verification for postgres 10")
-		return
-	}
-
 	expectedSlots, err := testsUtils.GetExpectedReplicationSlotsOnPod(namespace, clusterName, pod.GetName(), env)
 	Expect(err).ToNot(HaveOccurred())
 
@@ -2525,11 +2519,6 @@ func AssertClusterReplicationSlotsAligned(
 	namespace,
 	clusterName string,
 ) {
-	// Replication slot high availability requires PostgreSQL 11 or above
-	if env.PostgresVersion == 10 {
-		Skip("Ignoring replication slots verification for postgres 10")
-	}
-
 	podList, err := env.GetClusterPodList(namespace, clusterName)
 	Expect(err).ToNot(HaveOccurred())
 	Eventually(func() bool {

tests/e2e/fastfailover_test.go

@@ -109,9 +109,6 @@ var _ = Describe("Fast failover", Serial, Label(tests.LabelPerformance, tests.La
 		// forcing a failover and measuring how much time passes between the
 		// last row written on timeline 1 and the first one on timeline 2.
 		It("can do a fast failover", func() {
-			if env.PostgresVersion == 10 {
-				Skip("replication slots not available in PostgreSQL 10 or older")
-			}
 			namespace = "primary-failover-time"
 			clusterName = "cluster-fast-failover"
 			// Create a cluster in a namespace we'll delete after the test

tests/e2e/fastswitchover_test.go

@@ -78,10 +78,6 @@ var _ = Describe("Fast switchover", Serial, Label(tests.LabelPerformance, tests.
 	})
 	Context("with HA Replication Slots", func() {
 		It("can do a fast switchover", func() {
-			if env.PostgresVersion == 10 {
-				Skip("replication slots are not available for PostgreSQL 10 or older")
-			}
-
 			// Create a cluster in a namespace we'll delete after the test
 			err := env.CreateNamespace(namespace)
 			Expect(err).ToNot(HaveOccurred())

tests/e2e/replication_slot_test.go

@@ -39,10 +39,6 @@ var _ = Describe("Replication Slot", Label(tests.LabelReplication), func() {
 		if testLevelEnv.Depth < int(level) {
 			Skip("Test depth is lower than the amount requested for this test")
 		}
-		if env.PostgresVersion == 10 {
-			Skip("Test will be skipped for PostgreSQL 10, replication slot " +
-				"high availability requires PostgreSQL 11 or above")
-		}
 	})
 
 	It("Can enable and disable replication slots", func() {

tests/e2e/scaling_test.go

@@ -42,10 +42,6 @@ var _ = Describe("Cluster scale up and down", Serial, Label(tests.LabelReplicati
 
 	Context("with HA Replication Slots", func() {
 		It("can scale the cluster size", func() {
-			if env.PostgresVersion == 10 {
-				Skip("replication slots are not available for PostgreSQL 10 or older")
-			}
-
 			// Create a cluster in a namespace we'll delete after the test
 			err := env.CreateNamespace(namespace)
 			Expect(err).ToNot(HaveOccurred())

tests/e2e/switchover_test.go

@@ -39,9 +39,6 @@ var _ = Describe("Switchover", Serial, Label(tests.LabelSelfHealing), func() {
 	})
 	Context("with HA Replication slots", func() {
 		It("reacts to switchover requests", func() {
-			if env.PostgresVersion == 10 {
-				Skip("replication slots not available for PostgreSQL 10 or older")
-			}
 			// Create a cluster in a namespace we'll delete after the test
 			err := env.CreateNamespace(namespace)
 			Expect(err).ToNot(HaveOccurred())