Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(docker): run with zeebe user #11786

Closed
wants to merge 1 commit into from
Closed

Conversation

megglos
Copy link
Contributor

@megglos megglos commented Feb 22, 2023

Description

Runs the zeebe process with the zeebe user instead of root.

I couldn't make use of USER zeebe right away as this would be breaking given a previous zeebe instance was running as root and thus file ownership being with root. I thus kept the initial user root but reapply file ownership to zeebe from the startup script and then switch to the zeebe user.

As a follow-up we could introduce the USER zeebe statement to the dockerfile and remove the added if block from the startup script. E.g. with zeebe 8.3.0.

The changes to the Dockerfile could be backported to 8.1/8.0 and would allow users to configure the zeebe user manually in a new setup and not have trouble with mounting the log directory due to this. I would need to verify this on the stable branches though.

Note:

Related issues

closes #11784

@github-actions
Copy link
Contributor

github-actions bot commented Feb 22, 2023

Test Results

   990 files  ±    0     990 suites  ±0   1h 46m 44s ⏱️ + 6m 59s
7 627 tests  - 219  7 618 ✔️  - 219  9 💤 ±0  0 ±0 
7 820 runs   - 219  7 811 ✔️  - 219  9 💤 ±0  0 ±0 

Results for commit 7711681. ± Comparison against base commit 38b5f48.

♻️ This comment has been updated with latest results.

@megglos megglos force-pushed the meg-11784-docker-user branch 4 times, most recently from 5de908b to 6a06c82 Compare February 23, 2023 09:10
Dockerfile Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker: Allow running the zeebe process with an unprivileged user
2 participants