zhangyue19921010 commented on a change in pull request #10686:
URL: https://github.com/apache/druid/pull/10686#discussion_r546495715
##########
File path: docs/development/extensions-core/kubernetes.md
##########
@@ -57,3 +55,34 @@ Additionally, this extension has following configuration.
|`druid.discovery.k8s.renewDeadline`|`Duration`|Lease renewal period used by
Leader.|PT17S|No|
|`druid.discovery.k8s.retryPeriod`|`Duration`|Retry wait used by Leader
Election algorithm on failed operations.|PT5S|No|
+### Gotchas
+
+- Label/Annotation path in each pod spec MUST EXIST, which is easily satisfied
if there is at least one label/annotation in the pod spec already. This
limitation may be removed in future.
+- Druid Pods need permissions to be able to add labels to self-pod, List and
Watch other Pods, create ConfigMap for leader election. Assuming, "default"
service account is used by Druid pods, you might need to add following or
something similar Kubernetes Role and Role Binding.
Review comment:
Just curious. Which kind of druid server needs these `add labels to
self-pod, List and Watch other Pods, create ConfigMap` permissions?
MiddleManager or all the druid servers.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
