-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Feature Alternatives to sudo for privilege escalation #67897
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
base: master
Are you sure you want to change the base?
Conversation
`sudo` is the default privilege escalation. `sudo_cmd` has been added and allows alternatives to be specified in the minion config file via this variable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want to add this as a new config option by adding it to salt/config/__init__.py
and document it in conf/minion
and doc/ref/configuration/minion.rst
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This also needs a changelog and a test
`sudo` is the default privilege escalation. `sudo_cmd` has been added and allows alternatives to be specified in the minion config file via this variable.
Added as a new config option by adding it to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This adds the sudo_cmd
config option, but I don't see where this is implemented later on in Salt
What does this PR do?
This commit adds functionality for alternative privilege escalation
commands to be used via the
sudo_cmd
variable in the minion configfile. If the variable is absent the default of
sudo
is used.What issues does this PR fix or reference?
Fixes #67255
Previous Behavior
Only
sudo
was available for privilege escalation.New Behavior
sudo
is the default privilege escalation.sudo_cmd
has been addedand allows alternatives to be specified in the minion config file via this
variable.
Merge requirements satisfied?
[NOTICE] Bug fixes or features added to Salt require tests.
Commits signed with GPG?
Yes