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

Creating the tf user has shifted the uid/gid of centos and all further users #342

Closed
mboisson opened this issue Jan 20, 2025 · 0 comments · Fixed by #343
Closed

Creating the tf user has shifted the uid/gid of centos and all further users #342

mboisson opened this issue Jan 20, 2025 · 0 comments · Fixed by #343
Assignees
Labels
enhancement New feature or request

Comments

@mboisson
Copy link
Member

mboisson commented Jan 20, 2025

The tf user that gets created since #316 is created with uid:1000, which means centos shifts from uid:1000/gid:1000 to uid:1001/gid:1001.
This means that a previous local user with uid:1001, which uses PR ComputeCanada/puppet-magic_castle#312 can no longer be created, because it fails with

Jan 20 20:58:38 mgmt1.int.archimedes-dev.c3.ca puppet-agent[1640]: GID 1001 already exists, use allowdupe to force group creation
Jan 20 20:58:38 mgmt1.int.archimedes-dev.c3.ca puppet-agent[1640]: (/Stage[main]/Profile::Users::Local/Profile::Users::Local_user[nixuser]/Group[nixuser]/ensure) change from 'absent' to 'present' failed: GID 1001 already exists, use allowdupe to force group creation

Since, in our case, that UID has a massive number of files on the filesystem, it really needs to keep the same UID as before, and especially, centos should not use the uid/gid 1001, since it's typically not a privileged uid/gid.

@cmd-ntrf cmd-ntrf transferred this issue from ComputeCanada/puppet-magic_castle Jan 20, 2025
@cmd-ntrf cmd-ntrf self-assigned this Jan 21, 2025
@cmd-ntrf cmd-ntrf added the enhancement New feature or request label Jan 21, 2025
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
None yet
Development

Successfully merging a pull request may close this issue.

2 participants