Explicit Group Init in Boot Config #137
Merged
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
* added license files and test environments to .gitignore * Added more license configuration options * Disable THP using tuned * new variables for version and build. updated to 9.0.0.1 * check if splunk_package_version is newer. regex gets full version * Fixed many boot-start issues
* Adding DMC Playbook and Task * DMC playbook merge with task - update README
Co-authored-by: Merrel Raney III <merrel.raney@fiscal.treasury.gov> Co-authored-by: dtwersky <80541771+dtwersky@users.noreply.github.com>
|
My bad. I tested it after the user was already created. This seems to work fine if the user does not exist. |
|
Awesome - thanks. |
|
Now I'm thinking if this should have been |
|
It definitely should be, but the issue with the splunk init boot config still requires a patch. I can add it to the PR if you'd like. |
dtwersky
approved these changes
Nov 8, 2022
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.
Summary
Had to rebase a bit since my commits weren't attributed to my user, but the only commit needed is the last one. Essentially I'm just adding the
-groupflag to the boot start command since it's not there currently, and that causes some drift from the expected configs of the file permissions.Changes
Changed roles/splunk/tasks/configure_splunk_boot.yml to include
-group {{ splunk_nix_group }}