opendev/system-config
Code Issues Proposed changes

Fix jeepyb install require on pypi.o.o.

Browse Source 
The jeepyb install exec requires mysql::python but does not include that
class. This results in an error on hosts that do not include
mysql::python in some other manifest eg on pypi.openstack.org.

Include mysql::python in jeepyb's init manifest so that this require
works regardless of its inclusion with other manifests.

Change-Id: Iad1b1c723280df97ee5aa32512b36cfe1dc7034e
Reviewed-on: https://review.openstack.org/17414
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
This commit is contained in:
Clark Boylan 2012-12-03 12:47:31 -08:00 committed by Jenkins
parent 4cfb8b346d
commit ded8a5ce88
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/jeepyb/manifests/init.pp
View File

@ -3,6 +3,7 @@
class jeepyb ( class jeepyb (
$git_source_repo = 'https://github.com/openstack-ci/jeepyb.git', $git_source_repo = 'https://github.com/openstack-ci/jeepyb.git',
) { ) {
include mysql::python
if ! defined(Package['python-paramiko']) { if ! defined(Package['python-paramiko']) {
package { 'python-paramiko': package { 'python-paramiko':
@ -49,5 +50,4 @@ class jeepyb (
require => Class['mysql::python'], require => Class['mysql::python'],
subscribe => Vcsrepo['/opt/jeepyb'], subscribe => Vcsrepo['/opt/jeepyb'],
} }
} }