sudo #1
Open
sudo #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
We shouldn't be setting anything here out of the box. If users want this they can set it themselves. This will be a major version bump. Signed-off-by: Tim Smith <[email protected]>
It was straight out of Debian Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Enable sudoers.d out of the box on Linux
Remove sysdadmin group from the default groups attribute
Signed-off-by: Tim Smith <[email protected]>
Switch to dokken images and test on Debian 9
Signed-off-by: Robinson, Patrick, NMD-A3 (Extern) <[email protected]>
…pe String always will be return true
FIX: Setenv ignored in command list user & group
Signed-off-by: Tim Smith <[email protected]>
Added a new `visudo_path` attribute to the sudo resource to allow users to specify the installed path of visudo. In addition, the setting for the PATH environment variable on SLES and Solaris does not include the sbin path in which the visudo command is installed. This ensures the correct path value is inserted into the PATH environment variable automatically before executing the command. Fixes chef-boneyard#116 Signed-off-by: Ryan Hass <[email protected]>
Improve visudo path resolution.
Signed-off-by: Tim Smith <[email protected]>
FIX: in templates the attribute "passwordless" and other with data ty…
Signed-off-by: michael.sgarbossa <[email protected]>
New attribute for defining /etc/sudoers.d mode
Signed-off-by: Tim Smith <[email protected]>
Don't use a node attribute to store the config_prefix information. Do it with the same case statement, but allow the user to override it. Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Steve Clark <[email protected]>
Update to support AIX
disables wiring up the resource on chef >= 14.0 and avoids deprecation warning. Signed-off-by: Lamont Granquist <[email protected]>
…sion-for-provides add chef_version_for_provides
also bonus fix for removing use of node.set in chefspec tests Signed-off-by: Lamont Granquist <[email protected]>
Signed-off-by: Lamont Granquist <[email protected]>
This reverts commit 2d2cf0a.
Signed-off-by: Lamont Granquist <[email protected]>
…sion-for-provides2 fix chef_version_for_provides
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
If you use a non absolute path (or a `Cmnd_Alias` which is represented in ALL_CAPS) the sudoers validation will fail it but not tell you why. This makes it easier/quicker to spot the issue. Signed-off-by: Ben Abrams <[email protected]>
…-commands-with-paths adds better validation for commands passed in
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
No need for any of this anymore Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Amal Soman <[email protected]>
Metadata syntax issue cookbook failed in chef:12
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
[Describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves]
Check List