Skip to content

Commit eb7b07d

Browse files
authored
add service account properly (#142)
### Changelog - primary site: fixed a bug introduced in v0.0.76 where the query service service account object not be created if enabled. ### Docs None. ### Description it looks like when renaming the stream service to query service, I didn't properly namespace the values used in the query-service serviceaccount template. ### Testing - [x] rendered template using values that include a service account, verify that the required service account + annotations are created. <!-- Describe the problem, what has changed, and motivation behind those changes. Pretend you are advocating for this change and the reader is skeptical. --> <!-- In addition to unit tests, describe any manual testing you did to validate this change. --> Fixes: FIRE-159 <table><tr><th>Before</th><th>After</th></tr><tr><td> <!--before content goes here--> </td><td> <!--after content goes here--> </td></tr></table> <!-- If necessary, link relevant Linear or Github issues. Use `Fixes: foxglove/repo#1234` to auto-close the Github issue or Fixes: FG-### for Linear isses. -->
1 parent 7b0a4ab commit eb7b07d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
{{- $values := include "primary-site.mergedQueryServiceValues" . | fromYaml -}}
2-
{{- if .enabled }}
2+
{{- if $values.deployment.serviceAccount.enabled }}
33
apiVersion: v1
44
kind: ServiceAccount
55
metadata:
66
name: query-service
77
annotations:
8-
{{- range $key, $value := $values.annotations }}
8+
{{- range $key, $value := $values.deployment.serviceAccount.annotations }}
99
{{ $key }}: {{ $value | quote }}
1010
{{- end }}
1111
{{- end }}

0 commit comments

Comments
 (0)