[incubator/druid] Add securityContext and fix local storage mounting path#23561
[incubator/druid] Add securityContext and fix local storage mounting path#23561k8s-ci-robot merged 3 commits intohelm:masterfrom AWaterColorPen:users/slyao/druid
Conversation
Signed-off-by: AWaterColorPen <hellis@foxmail.com>
Signed-off-by: AWaterColorPen <hellis@foxmail.com>
|
Hi @AWaterColorPen. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/assign @maver1ck |
|
@sameersbn |
|
/ok-to-test |
Signed-off-by: AWaterColorPen <hellis@foxmail.com>
|
/lgtm |
|
@AWaterColorPen: you cannot LGTM your own PR. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@maver1ck could you help for a |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: AWaterColorPen, maver1ck The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Is this a new chart
What this PR does / why we need it:
securityContextto Historical and MiddleManager statefulSetsecurityContext.fsGroup=1000in values.yaml, which is the default id of the druid container userdruid.HistoricalandMiddleManagernode's mount volume path from/var/druid/to/opt/druid/var/druid/, which is the default path to store stateful files.Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close that issue when PR gets merged)Special notes for your reviewer:
base on #23069 and #23121 PRs, but they blocked.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[stable/mychartname])