Skip to content

Commit

Permalink
Adding default vaults to values.yaml for query and selector services.
Browse files Browse the repository at this point in the history
Signed-off-by: Vishu Kamble <[email protected]>
  • Loading branch information
vishukamble committed Jul 24, 2024
1 parent 467bf8c commit e912cb4
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 5 deletions.
6 changes: 3 additions & 3 deletions charts/jaeger/templates/collector-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,9 @@ spec:
protocol: TCP
targetPort: {{ .Values.collector.service.otlp.http.name }}
{{- end }}
- name: {{ .Values.collector.service.portName | default "admin" }}
port: 14269
targetPort: {{ .Values.collector.service.targetPortName | default "admin}}
- name: {{ .Values.collector.service.admin.name }}
port: {{ .Values.collector.service.admin.port }}
targetPort: {{ .Values.collector.service.admin.targetPort }}
selector:
{{- include "jaeger.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: collector
Expand Down
4 changes: 2 additions & 2 deletions charts/jaeger/templates/query-svc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@ spec:
port: 16685
protocol: TCP
targetPort: grpc
- name: {{ .Values.query.service.portName | default "admin" }}
- name: {{ .Values.query.service.admin.name }}
port: 16687
protocol: TCP
targetPort: {{ .Values.query.service.targetPortName | default "admin" }}
targetPort: {{ .Values.query.service.admin.targetPort }}
selector:
{{- include "jaeger.selectorLabels" . | nindent 4 }}
app.kubernetes.io/component: query
Expand Down
9 changes: 9 additions & 0 deletions charts/jaeger/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -510,6 +510,11 @@ collector:
# name: otlp-http
# port: 4318
# nodePort:
admin:
{}
# name: "admin"
# port: 14269
# targetPort: "admin"
ingress:
enabled: false
# For Kubernetes >= 1.18 you should specify the ingress-controller via the field ingressClassName
Expand Down Expand Up @@ -693,6 +698,10 @@ query:
# targetPort: 8080
# Specify a specific node port when type is NodePort
# nodePort: 32500
admin:
{}
# name: admin
# targetPort: admin
ingress:
enabled: false
# For Kubernetes >= 1.18 you should specify the ingress-controller via the field ingressClassName
Expand Down

0 comments on commit e912cb4

Please sign in to comment.