Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(kubernetes): Improve the security of the kubernetes/helm charts #1782

Merged
merged 12 commits into from
Aug 20, 2020
Prev Previous commit
Next Next commit
1747 | remove hardcoding from parameters for gms host and port
  • Loading branch information
shakti-garg committed Aug 8, 2020
commit 15b73c6be47367542ac15f9f3f4a18aad940332a
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ spec:
port: http
env:
- name: DATAHUB_GMS_HOST
value: "{{ .Values.global.datahub.gms.host }}"
value: {{ printf "%s-%s" .Release.Name "datahub-gms" }}
- name: DATAHUB_GMS_PORT
value: "{{ .Values.global.datahub.gms.port }}"
- name: DATAHUB_SECRET
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,5 @@ datahub:
global:
datahub:
gms:
host: "datahub-gms-deployment"
port: "8080"
appVersion: "1.0"
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
apiVersion: v1
kind: Service
metadata:
name: {{ include "datahub-gms.fullname" . }}
name: {{ printf "%s-%s" .Release.Name "datahub-gms" }}
labels:
{{- include "datahub-gms.labels" . | nindent 4 }}
spec:
type: {{ .Values.service.type }}
ports:
- port: {{ .Values.service.port }}
- port: {{ .Values.global.datahub.gms.port }}
targetPort: http
protocol: TCP
name: http
Expand Down
2 changes: 0 additions & 2 deletions contrib/kubernetes/datahub/charts/datahub-gms/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ securityContext: {}

service:
type: LoadBalancer #ClusterIP
port: 8080

ingress:
enabled: false
Expand Down Expand Up @@ -101,7 +100,6 @@ global:

datahub:
gms:
host: "datahub-gms-service"
port: "8080"
appVersion: "1.0"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ spec:
- name: KAFKA_SCHEMAREGISTRY_URL
value: "{{ .Values.global.kafka.schemaregistry.url }}"
- name: GMS_HOST
value: "{{ .Values.global.datahub.gms.host }}"
value: {{ printf "%s-%s" .Release.Name "datahub-gms" }}
- name: GMS_PORT
value: "{{ .Values.global.datahub.gms.port }}"
resources:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ global:

datahub:
gms:
host: "datahub-gms-deployment"
port: "8080"

hostAliases:
Expand Down
1 change: 0 additions & 1 deletion contrib/kubernetes/datahub/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ global:

datahub:
gms:
host: "datahub-gms-deployment"
port: "8080"
appVersion: "1.0"

Expand Down