system-config/modules/openstack_project/manifests/params.pp
Colleen Murphy 79e04fedd7 Stop installing puppet with puppet
Currently, the openstack_project::server class unconditionally ensures
the 'puppet' package is installed. As long as we're using puppet 3, this
is a noop: if puppet is running, then it's already installled. However,
if we have installed puppet 4 from the puppetlabs repositories, then the
puppet 4 package is actually called 'puppet-agent', and reinstalling the
'puppet' package from the distro repositories will clobber it. We could
work around this in two ways: one is to add logic to the
openstack_project::server class to manage the correct package name and
correct package version based on a parameter or the current state of the
system, and the other is to stop managing it entirely from puppet. Since
we're already managing it from ansible, let's go with the
less-puppet-code option.

This change also lets us make the puppet-4 system-config-specific beaker
jobs voting.

Change-Id: I32c02302a5009dcd3a54fae7fb346f2214be886a
2018-07-20 09:24:31 +02:00

40 lines
1.1 KiB
Puppet

# Class: openstack_project::params
#
# This class holds parameters that need to be
# accessed by other classes.
class openstack_project::params {
$cross_platform_packages = [
'at',
'git',
'lvm2',
'parted',
'rsync',
'strace',
'tcpdump',
'wget',
]
case $::osfamily {
'RedHat': {
$packages = concat($cross_platform_packages, ['iputils', 'bind-utils'])
$user_packages = ['emacs-nox', 'vim-enhanced']
$login_defs = 'puppet:///modules/openstack_project/login.defs.redhat'
}
'Debian': {
$packages = concat($cross_platform_packages, ['iputils-ping', 'dnsutils'])
case $::operatingsystemrelease {
/^(12|14)\.(04|10)$/: {
$user_packages = ['emacs23-nox', 'vim-nox', 'iftop',
'sysstat', 'iotop']
}
default: {
$user_packages = ['emacs-nox', 'vim-nox']
}
}
$login_defs = 'puppet:///modules/openstack_project/login.defs.debian'
}
default: {
fail("Unsupported osfamily: ${::osfamily} The 'openstack_project' module only supports osfamily Debian or RedHat (slaves only).")
}
}
}