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

Make username configurable in the rpm SPEC file to avoid conflicts with existing salt user #66944

Open
upendar245 opened this issue Oct 3, 2024 · 1 comment
Labels
Feature new functionality including changes to functionality and code refactors, etc. needs-triage

Comments

@upendar245
Copy link

Description of Issue/Enhancement

Currently, the SaltStack SPEC file hardcodes the username salt for the service user. This can cause conflicts for users who already have an existing salt user account used for other purposes. It would be beneficial to make the username configurable during installation or provide a mechanism to override the default user.

Steps to Reproduce

  1. Install SaltStack RPM on a system that already has a salt user created for a different application or purpose.
  2. Observe that the RPM preinstall script fails or overwrites the existing salt user settings.

Suggested Solution

  1. Modify the SPEC file to accept an environment variable or a build parameter that allows specifying a custom username.
  2. Update the preinstall, postinstall, and preuninstall scriptlets to use this configurable username.
  3. Provide documentation on how to use the new configuration options.

This enhancement would provide much-needed flexibility and prevent potential conflicts with existing system users.

Additional Context

Making the username configurable is a standard practice to avoid conflicts in environments where usernames might overlap across different applications. Implementing this change will make SaltStack more adaptable to various deployment environments.

@upendar245 upendar245 added Feature new functionality including changes to functionality and code refactors, etc. needs-triage labels Oct 3, 2024
Copy link

welcome bot commented Oct 3, 2024

Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here’s some information that may help as you continue your Salt journey.
Please be sure to review our Code of Conduct. Also, check out some of our community resources including:

There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar.
If you have additional questions, email us at [email protected]. We’re glad you’ve joined our community and look forward to doing awesome things with you!

@upendar245 upendar245 changed the title Make username configurable in the SPEC file to avoid conflicts with existing salt user Make username configurable in the rpm SPEC file to avoid conflicts with existing salt user Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature new functionality including changes to functionality and code refactors, etc. needs-triage
Projects
None yet
Development

No branches or pull requests

1 participant