opendev/system-config
Code Issues Proposed changes

Explicitly install gettext on job workers

Browse Source 
Some providers include gettext preinstalled by default on their base
images, but others do not. Make this consistent by explicitly
installing it when not already present. Rackspace seems to have
recently ceased including it on their Ubuntu Precise images, which
precipitated this patch.

Change-Id: Idb86c805e2fc56d885081876642bd9d3dbf58434
This commit is contained in:
Jeremy Stanley 2014-03-10 13:49:14 +00:00
parent b690718e94
commit bfabbaf6ef
2 changed files with 3 additions and 0 deletions
modules/jenkins/manifests
params.ppslave.pp

2
modules/jenkins/manifests/params.pp

@ -27,6 +27,7 @@ class jenkins::params {
# packages needed by document translation
$gnome_doc_package = 'gnome-doc-utils'
$libtidy_package = 'libtidy'
$gettext_package = 'gettext'
# for keystone ldap auth integration
$libsasl_dev = 'cyrus-sasl-devel'
$nspr_dev_package = 'nspr-devel'
@ -110,6 +111,7 @@ class jenkins::params {
# packages needed by document translation
$gnome_doc_package = 'gnome-doc-utils'
$libtidy_package = 'libtidy-0.99-0'
$gettext_package = 'gettext'
# for keystone ldap auth integration
$libsasl_dev = 'libsasl2-dev'
$mysql_dev_package = 'libmysqlclient-dev'

1
modules/jenkins/manifests/slave.pp

@ -39,6 +39,7 @@ class jenkins::slave(
$::jenkins::params::docbook_xml_package, # for building openstack docs
$::jenkins::params::docbook5_xml_package, # for building openstack docs
$::jenkins::params::docbook5_xsl_package, # for building openstack docs
$::jenkins::params::gettext_package, # for msgfmt, used in translating manuals
$::jenkins::params::gnome_doc_package, # for generating translation files for docs
$::jenkins::params::graphviz_package, # for generating graphs in docs
$::jenkins::params::firefox_package, # for selenium tests