Run Zookeeper container with user 'zookeeper' instead of 'root'

pravega / zookeeper-operator

Kubernetes Operator for Zookeeper

Apache License 2.0

366 stars 203 forks source link

Run Zookeeper container with user 'zookeeper' instead of 'root' #561

Open janhoy opened 1 year ago

janhoy commented 1 year ago

Change log description

Makes Zookeeper container run as user zookeeper instead of as 'root'

Purpose of the change

Fixes #538

What the code does

Changes user in Dockerfile

How to verify it

Build the image and run Zookeeper. Verify that it runs as user zookeeper.

TODO: The zookeeper POD needs fsGroup: 1000 for this to work, so the mounted data PVC with be owned by the zookeeper group.

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (9d283c2) 85.12% compared to head (137ed8f) 85.12%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #561 +/- ## ======================================= Coverage 85.12% 85.12% ======================================= Files 12 12 Lines 1613 1613 ======================================= Hits 1373 1373 Misses 155 155 Partials 85 85 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.