Skip to content

Commit

Permalink
🐛 fix(tracing): Add missing default env variables to services
Browse files Browse the repository at this point in the history
  • Loading branch information
MfCrizz committed Jul 11, 2023
1 parent 560213f commit 10373d5
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 10 deletions.
2 changes: 0 additions & 2 deletions chart/templates/ad-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ spec:
value: {{ quote .Values.adService.deployment.container.env.USER_AUTH_SERVICE_ADDRESS }}
- name: JAEGER_SERVICE_NAME
value: {{quote (printf "unguard-%s" .Values.adService.name) }}
{{if .Values.tracing.enabled}}
- name: JAEGER_AGENT_HOST
value: {{ quote (printf "%s-%s" .Values.jaeger.name .Values.adService.deployment.container.env.JAEGER_AGENT_HOST) }}
- name: JAEGER_SAMPLER_TYPE
Expand All @@ -75,4 +74,3 @@ spec:
value: {{ quote .Values.adService.deployment.container.env.JAEGER_SAMPLER_PARAM }}
- name: JAEGER_DISABLED
value: {{ quote .Values.adService.deployment.container.env.JAEGER_DISABLED }}
{{end}}
2 changes: 0 additions & 2 deletions chart/templates/frontend.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,6 @@ spec:
value: {{ quote .Values.frontend.deployment.container.env.MEMBERSHIP_SERVICE_BASE_PATH }}
- name: STATUS_SERVICE_BASE_PATH
value: {{ quote .Values.frontend.deployment.container.env.STATUS_SERVICE_BASE_PATH }}
{{if .Values.tracing.enabled}}
- name: JAEGER_SERVICE_NAME
value: {{quote (printf "unguard-%s" .Values.frontend.name) }}
- name: JAEGER_AGENT_HOST
Expand All @@ -90,4 +89,3 @@ spec:
value: {{ quote .Values.frontend.deployment.container.env.JAEGER_SAMPLER_PARAM }}
- name: JAEGER_DISABLED
value: {{ quote .Values.frontend.deployment.container.env.JAEGER_DISABLED }}
{{end}}
2 changes: 0 additions & 2 deletions chart/templates/microblog-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ spec:
value: {{ quote .Values.microblogService.deployment.container.env.REDIS_SERVICE_ADDRESS }}
- name: USER_AUTH_SERVICE_ADDRESS
value: {{ quote .Values.microblogService.deployment.container.env.USER_AUTH_SERVICE_ADDRESS }}
{{if .Values.tracing.enabled}}
- name: OPENTRACING_JAEGER_ENABLED
value: {{ quote .Values.microblogService.deployment.container.env.OPENTRACING_JAEGER_ENABLED }}
- name: JAEGER_SERVICE_NAME
Expand All @@ -74,4 +73,3 @@ spec:
value: {{ quote .Values.microblogService.deployment.container.env.JAEGER_SAMPLER_TYPE }}
- name: JAEGER_SAMPLER_PARAM
value: {{ quote .Values.microblogService.deployment.container.env.JAEGER_SAMPLER_PARAM }}
{{end}}
2 changes: 0 additions & 2 deletions chart/templates/proxy-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ spec:
env:
- name: SERVER_PORT
value: {{ quote .Values.proxyService.deployment.container.env.SERVER_PORT }}
{{if .Values.tracing.enabled}}
- name: OPENTRACING_JAEGER_ENABLED
value: {{ quote .Values.proxyService.deployment.container.env.OPENTRACING_JAEGER_ENABLED }}
- name: JAEGER_SERVICE_NAME
Expand All @@ -103,4 +102,3 @@ spec:
value: {{ quote .Values.proxyService.deployment.container.env.JAEGER_SAMPLER_TYPE }}
- name: JAEGER_SAMPLER_PARAM
value: {{ quote .Values.proxyService.deployment.container.env.JAEGER_SAMPLER_PARAM }}
{{end}}
2 changes: 0 additions & 2 deletions chart/templates/user-auth-service.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@ spec:
key: {{ tpl .Values.userAuthService.deployment.container.env.MARIADB_PASSWORD.secretKeyRef.key . }}
- name: JAEGER_SERVICE_NAME
value: {{quote (printf "unguard-%s" .Values.userAuthService.name) }}
{{if .Values.tracing.enabled}}
- name: JAEGER_AGENT_HOST
value: {{quote (printf "%s-%s" .Values.jaeger.name .Values.userAuthService.deployment.container.env.JAEGER_AGENT_HOST) }}
- name: JAEGER_SAMPLER_TYPE
Expand All @@ -77,4 +76,3 @@ spec:
value: {{ quote .Values.userAuthService.deployment.container.env.JAEGER_SAMPLER_PARAM }}
- name: JAEGER_DISABLED
value: {{ quote .Values.userAuthService.deployment.container.env.JAEGER_DISABLED }}
{{end}}

0 comments on commit 10373d5

Please sign in to comment.