chore: removed cliConnectPort from postgres

Author: akshat5302

charts/plane-ce/README.md

@@ -79,7 +79,6 @@
 | postgres.local_setup | true |  | Plane uses `postgres` as the primary database to store all the transactional data. This database can be hosted within kubernetes as part of helm chart deployment or can be used as hosted service remotely (e.g. aws rds or similar services). Set this to  `true` when you choose to setup stateful deployment of `postgres`. Mark it as `false` when using a remotely hosted database |
 | postgres.image | postgres:15.7-alpine |  | Using this key, user must provide the docker image name to setup the stateful deployment of `postgres`. (must be set when `postgres.local_setup=true`)|
 | postgres.servicePort | 5432 |  | This key sets the default port number to be used while setting up stateful deployment of `postgres`. |
-| postgres.cliConnectPort |  | | If you intend to access the hosted stateful deployment of postgres using any of the client tools (e.g Postico), this key helps you expose the port. The mentioned port must not be occupied by any other applicaiton |
 | postgres.volumeSize | 5Gi |  | While setting up the stateful deployment, while creating the persistant volume, volume allocation size need to be provided. This key helps you set the volume allocation size. Unit of this value must be in Mi (megabyte) or Gi (gigabyte) |
 | env.pgdb_username | plane |  | Database credentials are requried to access the hosted stateful deployment of `postgres`.  Use this key to set the username for the stateful deployment. |
 | env.pgdb_password | plane |  | Database credentials are requried to access the hosted stateful deployment of `postgres`.  Use this key to set the password for the stateful deployment. |

charts/plane-ce/questions.yml

@@ -317,10 +317,6 @@ questions:
     type: int
     default: 5432
     show_if: "postgres.local_setup=true"
-  - variable: postgres.cliConnectPort
-    label: CLI Connect Port
-    type: int
-    show_if: "postgres.local_setup=true"
   - variable: postgres.storageClass
     label: "Storage Class"
     type: string

charts/plane-ce/templates/workloads/postgres.stateful.yaml

@@ -19,30 +19,6 @@ spec:
   selector:
     app.name: {{ .Release.Namespace }}-{{ .Release.Name }}-pgdb
 ---
-{{- if and .Values.postgres.cliConnectPort .Values.postgres.cliConnectPort | int }}
-apiVersion: v1
-kind: Service
-metadata:
-  namespace: {{ .Release.Namespace }}
-  name: {{ .Release.Name }}-pgdb-cli-connect
-spec:
-  externalTrafficPolicy: Cluster
-  internalTrafficPolicy: Cluster
-  ipFamilies:
-    - IPv4
-  ipFamilyPolicy: SingleStack
-  ports:
-    - name: pg-{{ .Values.postgres.cliConnectPort | default 30000 }}
-      nodePort: {{ .Values.postgres.cliConnectPort | default 30000 }}
-      port: 5432
-      protocol: TCP
-      targetPort: 5432
-  selector:
-    app.name: {{ .Release.Namespace }}-{{ .Release.Name }}-pgdb
-  sessionAffinity: None
-  type: NodePort
----
-{{- end }}
 
 apiVersion: apps/v1
 kind: StatefulSet

charts/plane-ce/values.yaml

@@ -38,7 +38,6 @@ postgres:
   local_setup: true
   image: postgres:15.7-alpine
   servicePort: 5432
-  cliConnectPort: ""
   storageClass: longhorn
   volumeSize: 1Gi
   pullPolicy: IfNotPresent

charts/plane-enterprise/README.md

@@ -108,7 +108,6 @@
 | services.postgres.local_setup | true |  | Plane uses `postgres` as the primary database to store all the transactional data. This database can be hosted within kubernetes as part of helm chart deployment or can be used as hosted service remotely (e.g. aws rds or similar services). Set this to  `true` when you choose to setup stateful deployment of `postgres`. Mark it as `false` when using a remotely hosted database |
 | services.postgres.image | registry.plane.tools/plane/postgres:15.7-alpine |  | Using this key, user must provide the docker image name to setup the stateful deployment of `postgres`. (must be set when `services.postgres.local_setup=true`)|
 | services.postgres.servicePort | 5432 |  | This key sets the default port number to be used while setting up stateful deployment of `postgres`. |
-| services.postgres.cliConnectPort |  | | If you intend to access the hosted stateful deployment of postgres using any of the client tools (e.g Postico), this key helps you expose the port. The mentioned port must not be occupied by any other applicaiton|
 | services.postgres.volumeSize | 2Gi |  | While setting up the stateful deployment, while creating the persistant volume, volume allocation size need to be provided. This key helps you set the volume allocation size. Unit of this value must be in Mi (megabyte) or Gi (gigabyte) |
 | env.pgdb_username | plane |  | Database credentials are requried to access the hosted stateful deployment of `postgres`.  Use this key to set the username for the stateful deployment. |
 | env.pgdb_password | plane |  | Database credentials are requried to access the hosted stateful deployment of `postgres`.  Use this key to set the password for the stateful deployment. |

charts/plane-enterprise/questions.yml

@@ -339,6 +339,11 @@ questions:
   group: "Postgres Setup"
   # show_subquestion_if: true
   subquestions:
+  - variable: services.postgres.image
+    label: "Docker Image"
+    type: string
+    default: "registry.plane.tools/plane/postgres:15.7-alpine"
+    show_if: "services.postgres.local_setup=true"
   - variable: services.postgres.pullPolicy
     label: "Postgres Pull Policy"
     type: enum
@@ -347,20 +352,13 @@ questions:
     - "IfNotPresent"
     - "Never"
     default: "IfNotPresent"
-  - variable: services.postgres.image
-    label: "Docker Image"
-    type: string
-    default: "registry.plane.tools/plane/postgres:15.7-alpine"
     show_if: "services.postgres.local_setup=true"
   - variable: services.postgres.servicePort
     label: Service Port
     type: int
     default: 5432
     show_if: "services.postgres.local_setup=true"
-  - variable: services.postgres.cliConnectPort
-    label: CLI Connect Port
-    type: int
-    show_if: "services.postgres.local_setup=true"
+  
   - variable: services.postgres.volumeSize
     label: "Volume Size"
     type: string

charts/plane-enterprise/templates/workloads/postgres.stateful.yaml

@@ -19,31 +19,6 @@ spec:
   selector:
     app.name: {{ .Release.Namespace }}-{{ .Release.Name }}-pgdb
 ---
-{{- if and .Values.services.postgres.cliConnectPort .Values.services.postgres.cliConnectPort | int }}
-apiVersion: v1
-kind: Service
-metadata:
-  namespace: {{ .Release.Namespace }}
-  name: {{ .Release.Name }}-pgdb-cli-connect
-spec:
-  externalTrafficPolicy: Cluster
-  internalTrafficPolicy: Cluster
-  ipFamilies:
-    - IPv4
-  ipFamilyPolicy: SingleStack
-  ports:
-    - name: pg-{{ .Values.services.postgres.cliConnectPort | default 30000 }}
-      nodePort: {{ .Values.services.postgres.cliConnectPort | default 30000 }}
-      port: 5432
-      protocol: TCP
-      targetPort: 5432
-  selector:
-    app.name: {{ .Release.Namespace }}-{{ .Release.Name }}-pgdb
-  sessionAffinity: None
-  type: NodePort
----
-{{- end }}
-
 apiVersion: apps/v1
 kind: StatefulSet
 metadata:

charts/plane-enterprise/values.yaml

@@ -40,7 +40,6 @@ services:
     local_setup: true
     image: registry.plane.tools/plane/postgres:15.7-alpine
     servicePort: 5432
-    cliConnectPort: ''
     volumeSize: 2Gi
     pullPolicy: IfNotPresent
     assign_cluster_ip: false