-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Launch Gitea with custom UID/GID for 'git' user (fixes #2286) #2791
Conversation
LGTM |
@sapk how did you test that? |
This method is used other image. You can test it (i haven't done it) by setting env variable at docker run. We could maybe provide a docker-copose.yml file now that the container is configurable by env variable. Another solution for this problem, is to not create the user in the image (Dockerfile) and do a useradd with fixed UID and GID at startup of container. |
Codecov Report
@@ Coverage Diff @@
## master #2791 +/- ##
=======================================
Coverage 26.85% 26.85%
=======================================
Files 89 89
Lines 17607 17607
=======================================
Hits 4728 4728
Misses 12193 12193
Partials 686 686 Continue to review full report at Codecov.
|
LGTM (Seen this in other docker images as well) |
Not a formal unit test, but see #2286 (comment) for a manual test that shows how both UID and GID are modified. |
@HenrikBengtsson Unit-testing this would be very hard TBH. Since that would require the contributor to have docker installed. And in out CI it would mean running docker-in-docker which is a pain 😞 |
Fixes #2286