Fix port naming in codeintel-db service spec (#275)

* Fix port naming in codeintel-db service spec

Ultimately this change is minor, but may prevent confusion.

The port name and targetPort name are not actually used for anything besides human readability to my knowledge and there are no known issues with the current manifest. This does bring things in line with the convention in use in codeinsights-db service file though, and will eliminate possible confusion by readers

[Codeinsights manifest](https://sourcegraph.com/github.com/sourcegraph/deploy-sourcegraph-helm@release/5.0/-/blob/charts/sourcegraph/templates/codeinsights-db/codeinsights-db.Service.yaml?L23:5)

* make all the port names postgres

* changelog

Author: DaedalusG

charts/sourcegraph/CHANGELOG.md

@@ -8,6 +8,7 @@ Use `**BREAKING**:` to denote a breaking change
 
 ## Unreleased
 
+- Port names for all the DBs are renamed to `postgres` to create consistency across the deployments [#275](https://github.com/sourcegraph/deploy-sourcegraph-helm/pull/275)
 
 ## 5.0.0
 

charts/sourcegraph/templates/codeinsights-db/codeinsights-db.Service.yaml

@@ -18,9 +18,9 @@ metadata:
   name: codeinsights-db
 spec:
   ports:
-  - name: codeinsights-db
+  - name: postgres
     port: 5432
-    targetPort: codeinsights-db
+    targetPort: postgres
   selector:
     {{- include "sourcegraph.selectorLabels" . | nindent 4 }}
     app: codeinsights-db

charts/sourcegraph/templates/codeinsights-db/codeinsights-db.StatefulSet.yaml

@@ -80,7 +80,7 @@ spec:
         terminationMessagePolicy: FallbackToLogsOnError
         ports:
         - containerPort: 5432
-          name: codeinsights-db
+          name: postgres
         {{- if not .Values.sourcegraph.localDevMode }}
         resources:
           {{- toYaml .Values.codeInsightsDB.resources | nindent 10 }}

charts/sourcegraph/templates/codeintel-db/codeintel-db.Service.yaml

@@ -18,9 +18,9 @@ metadata:
   name: codeintel-db
 spec:
   ports:
-  - name: pgsql
+  - name: postgres
     port: 5432
-    targetPort: pgsql
+    targetPort: postgres
   selector:
     {{- include "sourcegraph.selectorLabels" . | nindent 4 }}
     app: codeintel-db

charts/sourcegraph/templates/codeintel-db/codeintel-db.StatefulSet.yaml

@@ -91,7 +91,7 @@ spec:
           periodSeconds: 10
         ports:
         - containerPort: 5432
-          name: pgsql
+          name: postgres
         {{- if not .Values.sourcegraph.localDevMode }}
         resources:
           {{- toYaml .Values.codeIntelDB.resources | nindent 10 }}

charts/sourcegraph/templates/pgsql/pgsql.Service.yaml

@@ -18,9 +18,9 @@ metadata:
   name: pgsql
 spec:
   ports:
-  - name: pgsql
+  - name: postgres
     port: 5432
-    targetPort: pgsql
+    targetPort: postgres
   selector:
     {{- include "sourcegraph.selectorLabels" . | nindent 4 }}
     app: pgsql

charts/sourcegraph/templates/pgsql/pgsql.StatefulSet.yaml

@@ -91,7 +91,7 @@ spec:
         {{- end }}
         ports:
         - containerPort: 5432
-          name: pgsql
+          name: postgres
         {{- if not .Values.sourcegraph.localDevMode }}
         resources:
           {{- toYaml .Values.pgsql.resources | nindent 10 }}