opendev/system-config
Code Issues Proposed changes

Correct elastic-recheck Puppet dependencies

Browse Source 
* modules/elastic_recheck/files/elastic-recheck.init: Update the
daemon invocation now that it has an executable entrypoints wrapper.

* modules/elastic_recheck/manifests/init.pp: Correct the Puppet
dependency tree sufficiently to be able to apply the manifest
successfully.

Change-Id: Ib018564e66c29d73b30aaf50c9306dc6d67ba2b2
This commit is contained in:
Jeremy Stanley 2013-09-30 19:55:41 +00:00
parent e0427cfbb7
commit 942e5f1dfb
2 changed files with 30 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
modules/elastic_recheck/files/elastic-recheck.init
View File

@ -17,9 +17,8 @@
PATH=/sbin:/usr/sbin:/bin:/usr/bin
DESC="Elastic Recheck"
NAME=elastic-recheck
#TODO: rework the following two lines once we have executable entrypoints
DAEMON=/usr/bin/python
DAEMON_ARGS="/opt/$NAME/elastic_recheck/bot.py /etc/$NAME/$NAME.conf"
DAEMON=/usr/local/bin/$NAME
DAEMON_ARGS=/etc/$NAME/$NAME.conf
PIDFILE=/var/run/$NAME/$NAME.pid
SCRIPTNAME=/etc/init.d/$NAME
USER=recheck

50
modules/elastic_recheck/manifests/init.pp
View File

@ -34,10 +34,7 @@ class elastic_recheck (
home => '/var/run/elastic-recheck',
shell => '/bin/false',
gid => 'recheck',
require => [
Group['recheck'],
File['/var/run/elastic-recheck'],
],
require => Group['recheck'],
}
vcsrepo { '/opt/elastic-recheck':
@ -58,24 +55,27 @@ class elastic_recheck (
}
file { '/var/run/elastic-recheck':
ensure => directory,
mode => '0755',
owner => 'recheck',
group => 'recheck',
ensure => directory,
mode => '0755',
owner => 'recheck',
group => 'recheck',
require => User['recheck'],
}
file { '/var/log/elastic-recheck':
ensure => directory,
mode => '0755',
owner => 'recheck',
group => 'recheck',
ensure => directory,
mode => '0755',
owner => 'recheck',
group => 'recheck',
require => User['recheck'],
}
file { '/etc/elastic-recheck':
ensure => directory,
mode => '0755',
owner => 'recheck',
group => 'recheck',
ensure => directory,
mode => '0755',
owner => 'recheck',
group => 'recheck',
require => User['recheck'],
}
file { '/etc/elastic-recheck/elastic-recheck.conf':
@ -84,21 +84,26 @@ class elastic_recheck (
owner => 'recheck',
group => 'recheck',
content => template('elastic_recheck/elastic-recheck.conf.erb'),
require => File['/etc/elastic-recheck'],
}
file { '/etc/elastic-recheck/recheckwatchbot.yaml':
ensure => present,
mode => '0640',
owner => 'recheck',
group => 'recheck',
source => 'puppet:///modules/elastic_recheck/recheckwatchbot.yaml',
ensure => present,
mode => '0640',
owner => 'recheck',
group => 'recheck',
source => 'puppet:///modules/elastic_recheck/recheckwatchbot.yaml',
require => File['/etc/elastic-recheck'],
}
# TODO(clarkb) put queries.json somewhere else.
file { '/etc/elastic-recheck/queries.json':
ensure => link,
target => '/opt/elastic-recheck/queries.json',
require => Vcsrepo['/opt/elastic-recheck'],
require => [
Vcsrepo['/opt/elastic-recheck'],
File['/etc/elastic-recheck'],
],
}
file { $gerrit_ssh_private_key:
@ -107,6 +112,7 @@ class elastic_recheck (
owner => 'recheck',
group => 'recheck',
content => $gerrit_ssh_private_key_contents,
require => User['recheck'],
}
file { '/etc/init.d/elastic-recheck':