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.

Sign up for GitHub

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

Jump to bottom

Add a way to delete users/groups #738

Closed
bgilbert opened this issue Feb 22, 2019 · 1 comment · Fixed by #1014
Closed

Add a way to delete users/groups #738

bgilbert opened this issue Feb 22, 2019 · 1 comment · Fixed by #1014
Assignees
@sohankunkerkar
Labels
area/usability jira for syncing to jira kind/enhancement spec change Requires changes to the spec

Comments

@bgilbert
Copy link
Contributor

Feature Request

Environment

Any

Desired Feature

Add a shouldExist attribute for users and groups, allowing the removal of an existing user/group.

Other Information

This would allow the OS to ship with a default user (core, fedora, etc.) that can be removed by an Ignition config if desired.

I don't think this functionality is critical for 3.0.
This was referenced Feb 22, 2019
Open
Closed
@ajeddeloh ajeddeloh mentioned this issue Mar 29, 2019
Closed
@ajeddeloh ajeddeloh added the spec change Requires changes to the spec label Oct 11, 2019
@sohankunkerkar sohankunkerkar self-assigned this Jun 16, 2020
@sohankunkerkar sohankunkerkar added the jira for syncing to jira label Jun 18, 2020
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 6, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
09fe8ca 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 6, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
c230e08 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 6, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
0bffd0b 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 6, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
8e236cf 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition-dracut that referenced this issue Jul 6, 2020
@sohankunkerkar
dracut/30ignition: Add packages to delete users/groups
54092a4 
This PR adds two packages namely "groupdel" and "userdel"
to delete the exiting users/groups from the base OS.
xref: coreos/ignition#738
sohankunkerkar added a commit to sohankunkerkar/ignition-dracut that referenced this issue Jul 6, 2020
@sohankunkerkar
dracut/30ignition: Add packages to delete users/groups
7a96888 
This PR adds two packages namely `groupdel` and `userdel`
to delete the existing users/groups from the base OS.
xref: coreos/ignition#738
@sohankunkerkar sohankunkerkar mentioned this issue Jul 6, 2020
Closed
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 6, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
2a121dc 
Fixes coreos#738
This PR will add a way to delete existing users/groups
@sohankunkerkar sohankunkerkar mentioned this issue Jul 6, 2020
Merged
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 7, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
9e09fbe 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 7, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
481973c 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 8, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
4d3bed5 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 8, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
382048a 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 11, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
9cdb03d 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 13, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
d39f485 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 14, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
6752b16 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 16, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
1dea73e 
Fixes coreos#738
This PR will add a way to delete existing users/groups
@cgwalters
Copy link
Member

The only user we really ship with by default that system administrators should be removing (IMO) is core. We'd have to have a stronger debate about "system" users like zincati.

I also remember we had a debate about this in the past where I was arguing that if the sysadmin provides anything in users, we should omit the default core user.

sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 20, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
2d355cb 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 20, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
0c09dd8 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 21, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
dccbd4f 
Fixes coreos#738
This PR will add a way to delete existing users/groups
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 21, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
1330d49 
Fixes coreos#738
Some users may want to only add a custom user, and
remove the `core` user injected via the default
configuration. This PR will enable users to delete
the existing users/groups in a given distro
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 21, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
efdea59 
Fixes coreos#738
Some users may want to only add a custom user, and
remove the `core` user injected via the default
configuration. This PR will enable users to delete
the existing users/groups in a given distro
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 21, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
3fb575a 
Fixes coreos#738
Some users may want to only add a custom user, and
remove the `core` user injected via the default
configuration. This PR will enable users to delete
the existing users/groups in a given distro
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 22, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
a672339 
Fixes coreos#738
Some users may want to add a custom user, and remove
the `core` user injected via the default configuration.
This PR will enable users to delete the existing
users/groups in a given distro.
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 22, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
b2a5394 
Fixes coreos#738
Some users may want to add a custom user, and remove
the `core` user injected via the default configuration.
This PR will enable users to delete the existing
users/groups in a given distro.
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 22, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
328023b 
Fixes coreos#738
Some users may want to add a custom user, and remove
the `core` user injected via the default configuration.
This PR will enable users to delete the existing
users/groups in a given distro.
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 22, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
6b4251e 
Fixes coreos#738
Some users may want to add a custom user, and remove
the `core` user injected via the default configuration.
This PR will enable users to delete the existing
users/groups in a given distro.
sohankunkerkar added a commit to sohankunkerkar/ignition-dracut that referenced this issue Jul 23, 2020
@sohankunkerkar
dracut/30ignition: Add packages to delete users/groups
10b5535 
This PR adds two packages namely `groupdel` and `userdel`
to delete the existing users/groups from the base OS.
xref: coreos/ignition#738
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 24, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
e72ed82 
Fixes coreos#738
Some users may want to add a custom user, and remove
the `core` user injected via the default configuration.
This PR will enable users to delete the existing
users/groups in a given distro.
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 24, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
e0d57f5 
Fixes coreos#738
Some users may want to add a custom user, and remove
the `core` user injected via the default configuration.
This PR will enable users to delete the existing
users/groups in a given distro.
sohankunkerkar added a commit to sohankunkerkar/ignition that referenced this issue Jul 29, 2020
@sohankunkerkar
passwd: allow removing the existing users/groups
6ad5c06 
Fixes coreos#738
Some users may want to add a custom user, and remove
the `core` user injected via the default configuration.
This PR will enable users to delete the existing
users/groups in a given distro.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sohankunkerkar sohankunkerkar

Labels
area/usability jira for syncing to jira kind/enhancement spec change Requires changes to the spec
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

passwd: Add a way to delete users/groups sohankunkerkar/ignition
4 participants
@cgwalters @bgilbert @ajeddeloh @sohankunkerkar