diff --git a/modules/jenkins/manifests/params.pp b/modules/jenkins/manifests/params.pp index 2bb5e637df..c75764f7bd 100644 --- a/modules/jenkins/manifests/params.pp +++ b/modules/jenkins/manifests/params.pp @@ -51,7 +51,8 @@ class jenkins::params { $xslt_package = 'libxslt' $xvfb_package = 'xorg-x11-server-Xvfb' # For Tooz unit tests - $zookeeper_package = 'zookeeper-server' + # FIXME: No zookeeper packages on RHEL + #$zookeeper_package = 'zookeeper-server' $cgroups_package = 'libcgroup' if ($::operatingsystem == 'Fedora') { $cgroups_tools_package = 'libcgroup-tools' diff --git a/modules/jenkins/manifests/slave.pp b/modules/jenkins/manifests/slave.pp index 3ee5e32cee..3a71613d0e 100644 --- a/modules/jenkins/manifests/slave.pp +++ b/modules/jenkins/manifests/slave.pp @@ -62,7 +62,6 @@ class jenkins::slave( $::jenkins::params::unzip_package, $::jenkins::params::xslt_package, # for building openstack docs $::jenkins::params::xvfb_package, # for selenium tests - $::jenkins::params::zookeeper_package, # for tooz unit tests ] if ($bare == false) { @@ -113,6 +112,11 @@ class jenkins::slave( require => Package[$::jenkins::params::jdk_package], } + # For Tooz unit tests + package { $::jenkins::params::zookeeper_package: + ensure => present, + } + } default: { fail("Unsupported osfamily: ${::osfamily} The 'jenkins' module only supports osfamily Debian or RedHat (slaves only).")