Fix Postgres persistence values in the chart (#313)

Co-authored-by: Krithika Sundararajan <[email protected]>
(cherry picked from commit 2768165)

Author: 2 people

infra/charts/turing/Chart.yaml

@@ -1,5 +1,5 @@
 apiVersion: v1
 description: "Turing: ML Experimentation System"
 name: turing
-version: 0.2.16
+version: 0.2.17
 appVersion: v1.0.0

infra/charts/turing/README.md

@@ -1,7 +1,7 @@
 # turing
 
 ---
-![Version: 0.2.16](https://img.shields.io/badge/Version-0.2.16-informational?style=flat-square)
+![Version: 0.2.17](https://img.shields.io/badge/Version-0.2.17-informational?style=flat-square)
 ![AppVersion: v1.0.0](https://img.shields.io/badge/AppVersion-v1.0.0-informational?style=flat-square)
 
 Turing: ML Experimentation System
@@ -77,9 +77,9 @@ The following table lists the configurable parameters of the Turing chart and th
 | mlp.extraEnvs | list | computed value | List of extra environment variables to add to MLP API container |
 | mlp.postgresql | object | `{"containerPorts":{"postgresql":5432},"nameOverride":"postgresql-mlp"}` | Postgresql configuration to be applied to MLP's postgresql database deployment Reference: https://artifacthub.io/packages/helm/bitnami/postgresql/12.1.9#parameters |
 | mlp.postgresql.nameOverride | string | `"postgresql-mlp"` | Name of MLP's Postgresql deployment |
-| postgresql | object | `{"auth":{"database":"turing","password":"turing","username":"turing"},"containerPorts":{"postgresql":5432},"image":{"tag":"12.13.0"},"metrics":{"enabled":false,"serviceMonitor":{"enabled":false}},"persistence":{"enabled":true,"size":"10Gi"},"replication":{"applicationName":"turing","enabled":false,"numSynchronousReplicas":2,"password":"repl_password","slaveReplicas":2,"synchronousCommit":"on","user":"repl_user"},"resources":{"requests":{"cpu":"500m","memory":"256Mi"}}}` | Postgresql configuration to be applied to Turing's postgresql database deployment Reference: https://artifacthub.io/packages/helm/bitnami/postgresql/12.1.9#parameters |
+| postgresql | object | `{"auth":{"database":"turing","password":"turing","username":"turing"},"containerPorts":{"postgresql":5432},"image":{"tag":"12.13.0"},"metrics":{"enabled":false,"serviceMonitor":{"enabled":false}},"primary":{"persistence":{"enabled":true,"size":"10Gi"}},"replication":{"applicationName":"turing","enabled":false,"numSynchronousReplicas":2,"password":"repl_password","slaveReplicas":2,"synchronousCommit":"on","user":"repl_user"},"resources":{"requests":{"cpu":"500m","memory":"256Mi"}}}` | Postgresql configuration to be applied to Turing's postgresql database deployment Reference: https://artifacthub.io/packages/helm/bitnami/postgresql/12.1.9#parameters |
 | postgresql.auth.password | string | `"turing"` | Password for Turing Postgresql database |
-| postgresql.persistence.enabled | bool | `true` | Persist Postgresql data in a Persistent Volume Claim |
+| postgresql.primary.persistence.enabled | bool | `true` | Persist Postgresql data in a Persistent Volume Claim |
 | postgresql.resources | object | `{"requests":{"cpu":"500m","memory":"256Mi"}}` | Resources requests and limits for Turing database. This should be set according to your cluster capacity and service level objectives. Reference: https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/ |
 | sentry.dsn | string | `""` | Sentry DSN value used by both Turing API and Turing UI |
 | tags.db | bool | `true` | Specifies if Postgresql database needs to be installed together with Turing |

infra/charts/turing/charts/merlin-0.7.1.tgz

@@ -144,8 +144,8 @@ The following table lists the configurable parameters of the Turing chart and th
 | postgresql.image.tag | string | `"12.13.0"` |  |
 | postgresql.metrics.enabled | bool | `false` |  |
 | postgresql.metrics.serviceMonitor.enabled | bool | `false` |  |
-| postgresql.persistence.enabled | bool | `true` |  |
-| postgresql.persistence.size | string | `"10Gi"` |  |
+| postgresql.primary.persistence.enabled | bool | `true` |  |
+| postgresql.primary.persistence.size | string | `"10Gi"` |  |
 | postgresql.replication.applicationName | string | `"merlin"` |  |
 | postgresql.replication.enabled | bool | `false` |  |
 | postgresql.replication.numSynchronousReplicas | int | `2` |  |

infra/charts/turing/charts/mlp-1.0.1.tgz

@@ -144,9 +144,10 @@ postgresql:
     requests:
       memory: 256Mi
       cpu: 500m
-  persistence:
-    enabled: true
-    size: 10Gi
+  primary:
+    persistence:
+      enabled: true
+      size: 10Gi
   auth:
     username: merlin
     database: merlin

infra/charts/turing/subcharts/merlin/README.md

@@ -102,8 +102,8 @@ The following table lists the configurable parameters of the Turing chart and th
 | postgresql.image.tag | string | `"12.13.0"` |  |
 | postgresql.metrics.enabled | bool | `false` |  |
 | postgresql.metrics.serviceMonitor.enabled | bool | `false` |  |
-| postgresql.persistence.enabled | bool | `true` | Persist Postgresql data in a Persistent Volume Claim |
-| postgresql.persistence.size | string | `"10Gi"` |  |
+| postgresql.primary.persistence.enabled | bool | `true` | Persist Postgresql data in a Persistent Volume Claim |
+| postgresql.primary.persistence.size | string | `"10Gi"` |  |
 | postgresql.replication.applicationName | string | `"mlp"` | Replication Cluster application name. Useful for defining multiple replication policies |
 | postgresql.replication.enabled | bool | `false` |  |
 | postgresql.replication.numSynchronousReplicas | int | `2` | From the number of `slaveReplicas` defined above, set the number of those that will have synchronous replication NOTE: It cannot be > slaveReplicas |

infra/charts/turing/subcharts/merlin/values.yaml

@@ -90,10 +90,11 @@ postgresql:
     requests:
       memory: 256Mi
       cpu: 500m
-  persistence:
-    # -- Persist Postgresql data in a Persistent Volume Claim
-    enabled: true
-    size: 10Gi
+  primary:
+    persistence:
+      # -- Persist Postgresql data in a Persistent Volume Claim
+      enabled: true
+      size: 10Gi
   auth:
     username: mlp
     database: mlp

infra/charts/turing/subcharts/mlp/README.md

@@ -213,10 +213,11 @@ postgresql:
     requests:
       memory: 256Mi
       cpu: 500m
-  persistence:
-    # -- Persist Postgresql data in a Persistent Volume Claim
-    enabled: true
-    size: 10Gi
+  primary:
+    persistence:
+      # -- Persist Postgresql data in a Persistent Volume Claim
+      enabled: true
+      size: 10Gi
   auth:
     database: turing
     username: turing

infra/charts/turing/subcharts/mlp/values.yaml

@@ -46,8 +46,9 @@ turing:
                 route_name: control
 
 postgresql: &postgresql
-  persistence:
-    enabled: false
+  primary:
+    persistence:
+      enabled: false
 
 merlin:
   postgresql: *postgresql