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

This commit sets the splunk_user for Windows to 'administrator' #107

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

ralfbosz
Copy link
Contributor

@ralfbosz ralfbosz commented Apr 18, 2017

There was only one 'default' which was 'root', but this user is
non-existent on Windows.

@ralfbosz ralfbosz changed the title This commit makes the splunk_user 'configurable' This commit sets the splunk_user for Windows to 'administrator' Apr 18, 2017
@bastelfreak
Copy link
Member

HI @ralfbosz, thanks for the commit. Please take a look at the used email address in your commit. It isn't asociated with your github account.

There was only one 'default' which was 'root', but this user is
non-existent on Windows
@ralfbosz
Copy link
Contributor Author

fixed it...

@bastelfreak bastelfreak merged commit c59a103 into voxpupuli:master Apr 19, 2017
@vinzent
Copy link
Contributor

vinzent commented Apr 19, 2017

@bastelfreak @ralfbosz is Administrator the correct value for this?

according the README the splunk_user param is the user to run Splunk as.. On Windows sytstems this isn't Administrator normally. I think the default is LocalSystem or something like this.

splunk_user will also set the ownership of some files to Administrator. I don't know if this is correct.

see also: https://docs.splunk.com/Documentation/Splunk/6.5.3/Installation/ChoosetheuserSplunkshouldrunas

@ralfbosz
Copy link
Contributor Author

The LocalSystem account is a predefined local account used by the service control manager. This account is not recognized by the security subsystem, so you cannot specify its name in a call to the LookupAccountName function.

https://msdn.microsoft.com/nl-nl/library/windows/desktop/ms684190(v=vs.85).aspx

On a Windows 2012 R2-server:

C:\windows\system32>puppet resource user localsystem
user { 'localsystem':
ensure => 'absent',
}

Seems to me that user isn't usable in Puppet. A better solution would be making the user 'configurable'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants