opendev/system-config
Code Issues Proposed changes

Ensure destination dir for bcpg link is present.

Browse Source 
The destination dir for the bcpg link needs to be present before the
link can be made. Add that dir to the gerrit init manifest and require
it in the link file resource.

Change-Id: I462cc96dcd0eafa814e3e3599a96eacc64665bcf
Reviewed-on: https://review.openstack.org/14319
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
This commit is contained in:
Clark Boylan 2012-10-10 18:01:29 -07:00 committed by Jenkins
parent 76a6e4cf9e
commit 3a6437a21b
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
modules/gerrit/manifests/init.pp
View File

@ -178,6 +178,12 @@ class gerrit($vhost_name=$fqdn,
require => File['/home/gerrit2/review_site'],
}
file { '/home/gerrit2/review_site/lib':
ensure => directory,
owner => 'gerrit2',
require => File['/home/gerrit2/review_site'],
}
# Skip replication if we're in test mode
if ($testmode == false) {
file { '/home/gerrit2/review_site/etc/replication.config':
@ -446,7 +452,10 @@ class gerrit($vhost_name=$fqdn,
file { '/home/gerrit2/review_site/lib/bcpg.jar':
ensure => link,
target => '/usr/share/java/bcpg.jar',
require => Package['libbcpg-java'],
require => [
Package['libbcpg-java'],
File['/home/gerrit2/review_site/lib'],
],
}
file { '/home/gerrit2/review_site/etc/contact_information.pub':
ensure => present,