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

[BUG] salt user HOME is in /home #64141

Closed
OrangeDog opened this issue Apr 24, 2023 · 1 comment · Fixed by #64174
Closed

[BUG] salt user HOME is in /home #64141

OrangeDog opened this issue Apr 24, 2023 · 1 comment · Fixed by #64174
Assignees
@dwoz
Labels
Bug broken, incorrect, or confusing behavior Packaging Related to packaging of Salt, not Salt's support for package management.
Milestone
Sulfur v3006.1

Comments

@OrangeDog
Copy link
Contributor

OrangeDog commented Apr 24, 2023 •
edited
Loading

Description
The new 3006 salt-master installers create a salt user to run the service as. However, this user is given a HOME of /home/salt, as if it were a human user.

Steps to Reproduce the behavior
Install 3006 via onedir package on Linux system

Expected behavior
The salt user should be a system user with a HOME of ideally /var/opt/saltstack.

Additional context
https://refspecs.linuxfoundation.org/FHS_3.0/fhs-3.0.html#optAddonApplicationSoftwarePackages
@OrangeDog OrangeDog added Bug broken, incorrect, or confusing behavior needs-triage labels Apr 24, 2023
@DaAwesomeP DaAwesomeP mentioned this issue Apr 24, 2023
Open
9 tasks
@OrangeDog OrangeDog added the Packaging Related to packaging of Salt, not Salt's support for package management. label Apr 25, 2023
@dwoz dwoz removed the needs-triage label Apr 25, 2023
@anilsil anilsil added this to the Sulfur v3006.1 milestone Apr 25, 2023
@dwoz dwoz assigned dwoz and unassigned Ch3LL Apr 25, 2023
@dwoz dwoz mentioned this issue Apr 29, 2023
Merged
6 tasks
dwoz added a commit to dwoz/salt that referenced this issue May 2, 2023
@dwoz
Add chahgelogs for saltstack#64141 and saltstack#64158
3822ee1
dwoz added a commit to dwoz/salt that referenced this issue May 2, 2023
@dwoz
Add chahgelogs for saltstack#64141 and saltstack#64158
173d476
dwoz added a commit to dwoz/salt that referenced this issue May 2, 2023
@dwoz
Add chahgelogs for saltstack#64141 and saltstack#64158
460670b
dwoz added a commit to dwoz/salt that referenced this issue May 2, 2023
@dwoz
Add chahgelogs for saltstack#64141 and saltstack#64158
8016284
dwoz added a commit to dwoz/salt that referenced this issue May 2, 2023
@dwoz
Add chahgelogs for saltstack#64141 and saltstack#64158
b223395
dwoz added a commit to dwoz/salt that referenced this issue May 3, 2023
@dwoz
Add chahgelogs for saltstack#64141 and saltstack#64158
ba4ecc5
@s0undt3ch s0undt3ch linked a pull request May 3, 2023 that will close this issue
[3006.x] Run salt minion with salt group permissions #64174
Merged
6 tasks
s0undt3ch pushed a commit to dwoz/salt that referenced this issue May 3, 2023
@dwoz @s0undt3ch
Add chahgelogs for saltstack#64141 and saltstack#64158
91eed27
garethgreenaway pushed a commit that referenced this issue May 3, 2023
@dwoz @garethgreenaway
Add chahgelogs for #64141 and #64158
28d87c1
@dwoz
Copy link
Contributor

dwoz commented May 3, 2023

Fixed by #64174

@dwoz dwoz closed this as completed May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dwoz dwoz

Labels
Bug broken, incorrect, or confusing behavior Packaging Related to packaging of Salt, not Salt's support for package management.
Projects
None yet
Milestone
Sulfur v3006.1
Development

Successfully merging a pull request may close this issue.

[3006.x] Run salt minion with salt group permissions dwoz/salt
4 participants
@OrangeDog @dwoz @anilsil @Ch3LL