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

Add solaris sparc parameters. #10

Merged
merged 1 commit into from
Dec 18, 2014
Merged

Conversation

nanliu
Copy link
Contributor

@nanliu nanliu commented Jul 30, 2014

No description provided.

@nanliu
Copy link
Contributor Author

nanliu commented Jul 31, 2014

The Solaris 10 splunk package does not create the required user/group despite a requirement for the pkg. See SPL-74427 for more info.

# pkgchk -l splunkforwarder

Pathname: /opt/splunkforwarder/bin
Type: directory
Expected mode: 0555
Expected owner: splunk
Expected group: splunk
Referenced by the following packages:
        splunkforwarder

This is a note that this patch alone won't completely solve the installation issue. I don't have a good solution to this bug and simply deferred it to the profile::splunk::forwarder class:

# Splunk forwarder wrapper.
class profile::splunk::forwarder (
  $manage_user = true,
  $uid = '200',
  $gid = '200',
) {
  include '::splunk::forwarder'

  user { 'splunk':
    ensure => present,
    uid    => $uid,
    gid    => $gid,
    home   => '/opt/splunkforwarder',
    shell  => '/bin/bash',
    before => Package[$::splunk::forwarder::package_name],
  }

  group { 'splunk':
    ensure => present,
    gid    => $gid,
  }
}

@puppetcla
Copy link

CLA signed by all contributors.

underscorgan pushed a commit that referenced this pull request Dec 18, 2014
Add solaris sparc parameters.
@underscorgan underscorgan merged commit 26fc975 into voxpupuli:master Dec 18, 2014
@underscorgan
Copy link
Contributor

thanks @nanliu

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.

3 participants