Protect install of every jeepyb package.
Because puppet sucks at installing packages, wrap the installation of everything that jeepyb needs in both jeepyb and in other places with if blocks. *facepalm* Change-Id: I69c73dabc3ea5d68bd057141b1ad758cdd68eafc Reviewed-on: https://review.openstack.org/17130 Reviewed-by: James E. Blair <corvus@inaugust.com> Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Approved: Monty Taylor <mordred@inaugust.com> Tested-by: Jenkins
This commit is contained in:
parent
52db16762b
commit
5c5033cbc7
@ -3,13 +3,17 @@
|
|||||||
class jeepyb (
|
class jeepyb (
|
||||||
$git_source_repo = 'https://github.com/openstack-ci/jeepyb.git',
|
$git_source_repo = 'https://github.com/openstack-ci/jeepyb.git',
|
||||||
) {
|
) {
|
||||||
$packages = [
|
|
||||||
'python-mysqldb',
|
|
||||||
'python-paramiko',
|
|
||||||
]
|
|
||||||
|
|
||||||
package { $packages:
|
if ! defined(Package['python-mysqldb']) {
|
||||||
ensure => present,
|
package { 'python-mysqldb':
|
||||||
|
ensure => present,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if ! defined(Package['python-paramiko']) {
|
||||||
|
package { 'python-paramiko':
|
||||||
|
ensure => present,
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if ! defined(Package['Pygithub']) {
|
if ! defined(Package['Pygithub']) {
|
||||||
|
@ -7,8 +7,7 @@ class lodgeit {
|
|||||||
'python-werkzeug',
|
'python-werkzeug',
|
||||||
'python-simplejson',
|
'python-simplejson',
|
||||||
'python-pygments',
|
'python-pygments',
|
||||||
'drizzle',
|
'drizzle']
|
||||||
'python-mysqldb' ]
|
|
||||||
|
|
||||||
include apache
|
include apache
|
||||||
|
|
||||||
@ -24,6 +23,12 @@ class lodgeit {
|
|||||||
ensure => present,
|
ensure => present,
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if ! defined(Package['python-mysqldb']) {
|
||||||
|
package { 'python-mysqldb':
|
||||||
|
ensure => present,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
package { 'SQLAlchemy':
|
package { 'SQLAlchemy':
|
||||||
ensure => present,
|
ensure => present,
|
||||||
provider => pip,
|
provider => pip,
|
||||||
|
@ -15,7 +15,6 @@ class zuul (
|
|||||||
'python-webob',
|
'python-webob',
|
||||||
'python-daemon',
|
'python-daemon',
|
||||||
'python-lockfile',
|
'python-lockfile',
|
||||||
'python-paramiko',
|
|
||||||
'python-paste',
|
'python-paste',
|
||||||
]
|
]
|
||||||
|
|
||||||
@ -31,6 +30,12 @@ class zuul (
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if ! defined(Package['python-paramiko']) {
|
||||||
|
package { 'python-paramiko':
|
||||||
|
ensure => present,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
# Packages that need to be installed from pip
|
# Packages that need to be installed from pip
|
||||||
$pip_packages = [
|
$pip_packages = [
|
||||||
'GitPython',
|
'GitPython',
|
||||||
|
Loading…
x
Reference in New Issue
Block a user