etc_dir should be configurable

[jaredledvina]

Description of problem

We're customizing the configuration directory of sensu and want to be able to set the etc_dir variable to a custom value. Looks like right now it's hard coded based on the OS type.

This is the section I'm looking at: https://github.com/sensu/sensu-puppet/blob/master/manifests/init.pp#L525-L549

Platform and version information

• Your OS: CentOS 6
• Your Ruby version:
• Your version of Puppet: Open Source Puppet 3.8
• Your version of Sensu: 0.26.5
• Your version of this module: Latest

[jaxxstorm]

Agreed, a PR would be appreciated!

[Phil-Friderici]

This is the section jaredledvina was looking at the 4th November 2016:

sensu-puppet/manifests/init.pp

Lines 525 to 549 in 93b2c2b

	case $::osfamily {
	'Debian','RedHat': {
	$etc_dir = '/etc/sensu'
	$conf_dir = "${etc_dir}/conf.d"
	$user = 'sensu'
	$group = 'sensu'
	$home_dir = '/opt/sensu'
	$shell = '/bin/false'
	$dir_mode = '0555'
	$file_mode = '0440'
	}
	'windows': {
	$etc_dir = 'C:/opt/sensu'
	$conf_dir = "${etc_dir}/conf.d"
	$user = 'Sensu'
	$group = "Users"
	$home_dir = $etc_dir
	$shell = undef
	$dir_mode = undef
	$file_mode = undef
	}
	default: {}
	}

[Phil-Friderici]

working on this in PR #741

[ghoneycutt]

Thanks for all your help!

Released in v2.28.0