Skip to content

Commit

Permalink
Expose the jmx port in service, when jmx is enabled. (confluentinc#261)
Browse files Browse the repository at this point in the history
* expose jmx exporter port when enabled

* change other charts as well

Co-authored-by: Moritz Schmitz von Hülst <[email protected]>
  • Loading branch information
Moritz Schmitz von Hülst and Moritz Schmitz von Hülst authored Sep 11, 2020
1 parent 5b13b07 commit 3434a1e
Show file tree
Hide file tree
Showing 6 changed files with 26 additions and 2 deletions.
4 changes: 4 additions & 0 deletions charts/cp-kafka-connect/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ spec:
ports:
- name: kafka-connect
port: {{ .Values.servicePort }}
{{- if .Values.prometheus.jmx.enabled }}
- name: metrics
port: {{ .Values.prometheus.jmx.port }}
{{- end }}
selector:
app: {{ template "cp-kafka-connect.name" . }}
release: {{ .Release.Name }}
4 changes: 4 additions & 0 deletions charts/cp-kafka-rest/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ spec:
ports:
- name: rest-proxy
port: {{ .Values.servicePort }}
{{- if .Values.prometheus.jmx.enabled }}
- name: metrics
port: {{ .Values.prometheus.jmx.port }}
{{- end }}
selector:
app: {{ template "cp-kafka-rest.name" . }}
release: {{ .Release.Name }}
6 changes: 5 additions & 1 deletion charts/cp-kafka/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ spec:
ports:
- port: 9092
name: broker
{{- if .Values.prometheus.jmx.enabled }}
- port: {{ .Values.prometheus.jmx.port }}
name: metrics
{{- end }}
selector:
app: {{ template "cp-kafka.name" . }}
release: {{ .Release.Name }}
release: {{ .Release.Name }}
4 changes: 4 additions & 0 deletions charts/cp-ksql-server/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ spec:
ports:
- name: ksql-server
port: {{ .Values.servicePort }}
{{- if .Values.prometheus.jmx.enabled }}
- name: metrics
port: {{ .Values.prometheus.jmx.port }}
{{- end }}
selector:
app: {{ template "cp-ksql-server.name" . }}
release: {{ .Release.Name }}
4 changes: 4 additions & 0 deletions charts/cp-schema-registry/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@ spec:
ports:
- name: schema-registry
port: {{ .Values.servicePort }}
{{- if .Values.prometheus.jmx.enabled }}
- name: metrics
port: {{ .Values.prometheus.jmx.port }}
{{- end }}
selector:
app: {{ template "cp-schema-registry.name" . }}
release: {{ .Release.Name }}
6 changes: 5 additions & 1 deletion charts/cp-zookeeper/templates/service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,10 @@ spec:
ports:
- port: {{ .Values.clientPort }}
name: client
{{- if .Values.prometheus.jmx.enabled }}
- name: metrics
port: {{ .Values.prometheus.jmx.port }}
{{- end }}
selector:
app: {{ template "cp-zookeeper.name" . }}
release: {{ .Release.Name }}
release: {{ .Release.Name }}

0 comments on commit 3434a1e

Please sign in to comment.