Skip to content

feat: no more special UID, only GID 0 - #197

Open
dotconfig404 wants to merge 2 commits into
OpenVoxProject:mainfrom
dotconfig404:feat/gid0-arbitrary-uid
Open

feat: no more special UID, only GID 0#197
dotconfig404 wants to merge 2 commits into
OpenVoxProject:mainfrom
dotconfig404:feat/gid0-arbitrary-uid

Conversation

@dotconfig404

@dotconfig404 dotconfig404 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Short description

As discussed in slack, this PR should make the access model much clearer. GID 0 is to be used for volumes and generally for file access, any UID can be used.
This does not fundamentally change anything that wasn't implemented already since
9e73538
but since the restructure of the images there was some regression, for example the addition of a script that sets permissions when a certain UID is used. Instead of making a specific UID special a fix should be applied across all UIDs the same.

Hopefully this PR will make it easier to maintain the repo.

Checklist

I have:

EDIT: wrong commit/PR mentioned

Signed-off-by: dotconfig404 <dotconfig@krutt.org>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@dotconfig404
dotconfig404 force-pushed the feat/gid0-arbitrary-uid branch from 0765bd9 to 86a2bff Compare August 14, 2026 14:07
Signed-off-by: Anthony Somerset <anthony@sts.io>

@anthonysomerset anthonysomerset left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks good to me, just a few markdown lint errors to clear

Comment thread MIGRATION.md

UID is changed from 1001 on alpine and 999 on ubuntu to 64604.
If you already deployed the containers with mounted volumes, you HAVE to change the ownership of these volumes and the files underneath.
The container runs with **group 0** and works under **any UID**. File access is granted exclusively through group 0, and the UID of files on mounted volumes does not matter. There is no `puppet` service account in the image anymore.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls do "one sentence one line" style.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also pls fix Trailing spaces

Comment thread MIGRATION.md
# Migrations

## V8.12.0 -> V8.13.0
## Coming from a fixed UID release

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls fix Trailing spaces

@rwaffen rwaffen added the enhancement New feature or request label Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants