opendev/system-config
Code Issues Proposed changes

Fix puppet subscription for elastic-recheck.

Browse Source 
The installation of elastic-recheck is done on multiple machines but the
irc bot is only run on one machine. Therefore don't notify the bot
service from the install exec instead subscribe to the exec from the
service.

Change-Id: I11f3d32e40408a02d948d0533f5a4c45c983adae
This commit is contained in:
Clark Boylan 2013-12-06 14:24:17 -08:00
parent 97322c888c
commit ff1e7bc208
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
modules/elastic_recheck/manifests/bot.pp
View File

@ -73,7 +73,10 @@ class elastic_recheck::bot (
service { 'elastic-recheck': service { 'elastic-recheck':
ensure => running, ensure => running,
enable => true, enable => true,
subscribe => File['/etc/elastic-recheck/elastic-recheck.conf'], subscribe => [
File['/etc/elastic-recheck/elastic-recheck.conf'],
Exec['::elastic_recheck::install_elastic-recheck'],
],
require => [ require => [
Class['elastic_recheck'], Class['elastic_recheck'],
File['/etc/init.d/elastic-recheck'], File['/etc/init.d/elastic-recheck'],

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

@ -41,7 +41,6 @@ class elastic_recheck (
command => 'pip install /opt/elastic-recheck', command => 'pip install /opt/elastic-recheck',
path => '/usr/local/bin:/usr/bin:/bin/', path => '/usr/local/bin:/usr/bin:/bin/',
refreshonly => true, refreshonly => true,
notify => Service['elastic-recheck'],
subscribe => Vcsrepo['/opt/elastic-recheck'], subscribe => Vcsrepo['/opt/elastic-recheck'],
require => Class['pip'], require => Class['pip'],
} }