Upgrade bitnami/postgresql subchart to 13.2.24 (apache#36156)

dabla · Dec 21, 2023 · 55f421b · 55f421b
1 parent f84eb2a
commit 55f421b
Show file tree

Hide file tree

Showing 8 changed files with 15 additions and 21 deletions.
diff --git a/chart/Chart.lock b/chart/Chart.lock
@@ -1,6 +1,6 @@
 dependencies:
 - name: postgresql
   repository: https://charts.bitnami.com/bitnami
-  version: 12.10.0
-digest: sha256:731562ef1f62ee687121df2d44ff8131a73aa63841f6cac858c30748ad349d55
-generated: "2023-08-25T13:23:48.02337-06:00"
+  version: 13.2.24
+digest: sha256:07f12ed410f106bf13eca69df16a1ef6690c4d4bfcb037943bbff6e71a22201d
+generated: "2023-12-09T17:23:53.209725+01:00"
diff --git a/chart/Chart.yaml b/chart/Chart.yaml
@@ -34,7 +34,7 @@ keywords:
   - scheduler
 dependencies:
   - name: postgresql
-    version: 12.10.0
+    version: 13.2.24
     repository: https://charts.bitnami.com/bitnami
     condition: postgresql.enabled
 maintainers:

diff --git a/chart/charts/postgresql-12.10.0.tgz b/chart/charts/postgresql-12.10.0.tgz
diff --git a/chart/charts/postgresql-13.2.24.tgz b/chart/charts/postgresql-13.2.24.tgz
diff --git a/chart/newsfragments/34817.significant.rst b/chart/newsfragments/34817.significant.rst
@@ -0,0 +1,8 @@
+The helm chart is now using a newer version of ``bitnami/postgresql`` dependency
+
+The version of ``bitnami/postgresql`` subchart upgraded from ``12.10.0`` to ``13.2.24``.
+The version of ``PostgreSQL`` binaries upgraded from ``11`` to ``16.1.0``.
+
+The change requires existing ``bitnami/postgresql`` subchart users to perform manual major version upgrade using ``pg_dumpall`` or ``pg_upgrade``.
+
+As a reminder, it is recommended to `set up an external database <https://airflow.apache.org/docs/helm-chart/stable/production-guide.html#database>`_ in production.
diff --git a/chart/values.schema.json b/chart/values.schema.json
@@ -5890,7 +5890,7 @@
                 "auth_type": {
                     "description": "Method of authenticating users",
                     "type": "string",
-                    "default": "md5"
+                    "default": "scram-sha-256"
                 },
                 "auth_file": {
                     "description": "The name of the file to load user names and passwords from",
@@ -6965,18 +6965,6 @@
                     "type": "boolean",
                     "default": true
                 },
-                "images": {
-                    "description": "PostgreSQL image values.",
-                    "type": "object",
-                    "additionalProperties": true,
-                    "properties": {
-                        "tag": {
-                            "description": "The PostgreSQL image tag.",
-                            "type": "string",
-                            "default": "11"
-                        }
-                    }
-                },
                 "auth": {
                     "description": "PostgreSQL authentication values.",
                     "type": "object",

diff --git a/chart/values.yaml b/chart/values.yaml
@@ -1895,7 +1895,7 @@ pgbouncer:
   command: ["pgbouncer", "-u", "nobody", "/etc/pgbouncer/pgbouncer.ini"]
   # Args to use for PgBouncer(templated).
   args: ~
-  auth_type: md5
+  auth_type: scram-sha-256
   auth_file: /etc/pgbouncer/users.txt
 
   # annotations to be added to the PgBouncer deployment
@@ -2277,8 +2277,6 @@ cleanup:
 # Not recommended for production
 postgresql:
   enabled: true
-  image:
-    tag: "11"
   auth:
     enablePostgresUser: true
     postgresPassword: postgres

diff --git a/helm_tests/other/test_pgbouncer.py b/helm_tests/other/test_pgbouncer.py
@@ -487,7 +487,7 @@ def test_auth_type_file_defaults(self):
         }
         ini = self._get_pgbouncer_ini(values)
 
-        assert "auth_type = md5" in ini
+        assert "auth_type = scram-sha-256" in ini
         assert "auth_file = /etc/pgbouncer/users.txt" in ini
 
     def test_auth_type_file_overrides(self):