feat: handling max_slot_wal_keep_size for pg 17 was needed as well

Author: samrose

ansible/files/admin_api_scripts/pg_upgrade_scripts/complete.sh

@@ -22,6 +22,18 @@ function cleanup {
 
     ship_logs "$LOG_FILE" || true
 
+    # Restore max_slot_wal_keep_size to 4096 only if both versions were PostgreSQL 17+
+    if [[ ("$OLD_PGVERSION" =~ ^17.* || "$OLD_PGVERSION" == "17-orioledb") && ("$PGVERSION" =~ ^17.* || "$PGVERSION" == "17-orioledb") ]]; then
+        sed -i 's/max_slot_wal_keep_size = -1/max_slot_wal_keep_size = 4096/' /etc/postgresql/postgresql.conf
+    fi
+
+    # Restart postgres to apply any configuration changes
+    if [ -z "$IS_CI" ]; then
+        systemctl restart postgresql
+    else
+        CI_start_postgres
+    fi
+
     exit "$EXIT_CODE"
 }
 

ansible/files/admin_api_scripts/pg_upgrade_scripts/initiate.sh

@@ -397,6 +397,11 @@ function initiate_upgrade {
     chown -R postgres:postgres "$MOUNT_POINT/"
     rm -rf "${PGDATANEW:?}/"
 
+    # Change max_slot_wal_keep_size to -1 for upgrade only if both versions are PostgreSQL 17+
+    if [[ ("$OLD_PGVERSION" =~ ^17.* || "$OLD_PGVERSION" == "17-orioledb") && ("$PGVERSION" =~ ^17.* || "$PGVERSION" == "17-orioledb") ]]; then
+        sed -i 's/max_slot_wal_keep_size = [0-9]*/max_slot_wal_keep_size = -1/' /etc/postgresql/postgresql.conf
+    fi
+
     if [ "$IS_NIX_UPGRADE" = "true" ]; then
         if [[ "$PGVERSION" =~ ^17.* || "$PGVERSION" == "17-orioledb" ]]; then
             LC_ALL=en_US.UTF-8 LC_CTYPE=en_US.UTF-8 LC_COLLATE=en_US.UTF-8 LANGUAGE=en_US.UTF-8 LANG=en_US.UTF-8 LOCALE_ARCHIVE=/usr/lib/locale/locale-archive su -c ". /nix/var/nix/profiles/default/etc/profile.d/nix-daemon.sh && $PGBINNEW/initdb --encoding=$SERVER_ENCODING --locale-provider=icu --icu-locale=en_US.UTF-8 -L $PGSHARENEW -D $PGDATANEW/ --username=supabase_admin" -s "$SHELL" postgres

ansible/vars.yml

@@ -9,9 +9,9 @@ postgres_major:
 
 # Full version strings for each major version
 postgres_release:
-  postgresorioledb-17: "17.0.1.078-orioledb-upgrade-3"
-  postgres17: "17.4.1.028-upgrade-3"
-  postgres15: "15.8.1.085-upgrade-3"
+  postgresorioledb-17: "17.0.1.078-orioledb-upgrade-4"
+  postgres17: "17.4.1.028-upgrade-4"
+  postgres15: "15.8.1.085-upgrade-4"
 
 # Non Postgres Extensions
 pgbouncer_release: "1.19.0"