abhishekagarwal87 commented on code in PR #13747:
URL: https://github.com/apache/druid/pull/13747#discussion_r1106663829
##########
helm/druid/README.md:
##########
@@ -199,6 +225,11 @@ The following table lists the configurable parameters of
the Druid chart and the
| `router.replicaCount` | router node replicas (deployment)
| `1` |
| `router.port` | port of router component
| `8888` |
| `router.serviceType` | service type for service
| `ClusterIP` |
+| `router.serviceAccount.create` | Create a service account for
router service | `true` |
+| `router.serviceAccount.name` | Service account name
| `` |
Review Comment:
what is the default service account name generated by helm? we should put
that here instead of an empty string as default.
##########
helm/druid/templates/historical/roleBinding.yaml:
##########
@@ -0,0 +1,20 @@
+{{- if and (.Values.rbac.create) (.Values.historical.enabled) }}
+apiVersion: rbac.authorization.k8s.io/v1
+kind: RoleBinding
+metadata:
+ name: {{ template "druid.historical.fullname" . }}
+ labels:
+ app: {{ include "druid.name" . }}
+ chart: {{ include "druid.chart" . }}
+ component: {{ .Values.historical.name }}
+ release: {{ .Release.Name }}
+ heritage: {{ .Release.Service }}
+roleRef:
+ apiGroup: rbac.authorization.k8s.io
+ kind: Role
+ name: {{ template "druid.historical.fullname" . }}
+subjects:
+ - kind: ServiceAccount
+ name: {{ .Values.historical.serviceAccount.name }}
Review Comment:
shouldn't this be `druid.historical.serviceAccountName`? The default value
of this property is empty. is the chart generating the k8s configs correctly?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
