opendev
/
puppet-zuul
Code Issues Proposed changes

Be conservative requiring python-yaml. 

A lot of modules require python-yaml. Be conservative requiring this
package to avoid conflicts when two or more of these modules end up
being used on the same host.

Change-Id: I35bf4f1070e6f248734c593e9277b737166904c1
Reviewed-on: https://review.openstack.org/13688
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
This commit is contained in:
Clark Boylan 2012-09-25 20:47:38 -07:00 committed by Jenkins
parent cbc2eac5c8
commit a57fac5480
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
manifests/init.pp
View File

@ -12,12 +12,20 @@ class zuul (
"python-lockfile",
"python-paramiko",
"python-paste",
"python-yaml"]
]
package { $packages:
ensure => "present",
}
# A lot of things need yaml, be conservative requiring this package to avoid
# conflicts with other modules.
if ! defined(Package['python-yaml']) {
package { 'python-yaml':
ensure => "present",
}
}
# Packages that need to be installed from pip
$pip_packages = ["GitPython"]