82c5863431
The postgresql init script on debian/ubuntu returns 0 no matter whether postgresql is started or stopped, so puppet has no way of knowing whether to start postgresql when the service is set to 'ensure => running'. This commit adds a param to the params class called $service_status which is set to the status piped to an egrep on debian/ubuntu, which reliably returns 0 if there are clusters running and 1 if there are none. The output before and after this patch can be seen below. If the init script is fixed at some point, the logic would need to be revisited.
Before patch postgresql remains stopped after puppet run.
$ # service postgresql stop
$ # * Stopping PostgreSQL 9.1 database server
$ # puppet apply -e "class {'postgresql::server':}"
$ # notice: Finished catalog run in 0.15 seconds
$ # service postgresql status
$ # Running clusters:
After patch postgresql is started after puppet run.
$ # service postgresql stop
$ # * Stopping PostgreSQL 9.1 database server
$ # puppet apply -e "class {'postgresql::server':}"
$ # notice: /Stage[main]/Postgresql::Server/Service[postgresqld]/ensure: ensure changed 'stopped' to 'running'
$ # notice: Finished catalog run in 2.26 seconds
$ # service postgresql status
$ # Running clusters: 9.1/main
91 lines
3.4 KiB
Puppet
91 lines
3.4 KiB
Puppet
# Class: postgresql::params
|
|
#
|
|
# The postgresql configuration settings.
|
|
#
|
|
# Parameters:
|
|
#
|
|
# Actions:
|
|
#
|
|
# Requires:
|
|
#
|
|
# Sample Usage:
|
|
#
|
|
class postgresql::params {
|
|
$user = 'postgres'
|
|
$group = 'postgres'
|
|
$ip_mask_deny_postgres_user = '0.0.0.0/0'
|
|
$ip_mask_allow_all_users = '127.0.0.1/32'
|
|
$listen_addresses = 'localhost'
|
|
# TODO: figure out a way to make this not platform-specific
|
|
$manage_redhat_firewall = false
|
|
|
|
# This is a bit hacky, but if the puppet nodes don't have pluginsync enabled,
|
|
# they will fail with a not-so-helpful error message. Here we are explicitly
|
|
# verifying that the custom fact exists (which implies that pluginsync is
|
|
# enabled and succeeded). If not, we fail with a hint that tells the user
|
|
# that pluginsync might not be enabled. Ideally this would be handled directly
|
|
# in puppet.
|
|
if ($::postgres_default_version == undef) {
|
|
fail "No value for postgres_default_version facter fact; it's possible that you don't have pluginsync enabled."
|
|
}
|
|
|
|
case $::operatingsystem {
|
|
default: {
|
|
$service_provider = undef
|
|
}
|
|
}
|
|
|
|
case $::osfamily {
|
|
'RedHat': {
|
|
$service_name = 'postgresql'
|
|
$client_package_name = 'postgresql'
|
|
$server_package_name = 'postgresql-server'
|
|
$needs_initdb = true
|
|
$initdb_path = '/usr/bin/initdb'
|
|
$createdb_path = '/usr/bin/createdb'
|
|
$psql_path = '/usr/bin/psql'
|
|
$datadir = '/var/lib/pgsql/data/'
|
|
$pg_hba_conf_path = '/var/lib/pgsql/data/pg_hba.conf'
|
|
$postgresql_conf_path = '/var/lib/pgsql/data/postgresql.conf'
|
|
$firewall_supported = true
|
|
$persist_firewall_command = '/sbin/iptables-save > /etc/sysconfig/iptables'
|
|
}
|
|
|
|
'Debian': {
|
|
case $::operatingsystem {
|
|
'Debian': {
|
|
$service_name = "postgresql"
|
|
}
|
|
|
|
'Ubuntu': {
|
|
case $::lsbmajdistrelease {
|
|
# thanks, ubuntu
|
|
'10': { $service_name = "postgresql-${::postgres_default_version}" }
|
|
default: { $service_name = "postgresql" }
|
|
}
|
|
}
|
|
}
|
|
|
|
$client_package_name = 'postgresql-client'
|
|
$server_package_name = 'postgresql'
|
|
$needs_initdb = false
|
|
$initdb_path = "/usr/lib/postgresql/${::postgres_default_version}/bin/initdb"
|
|
$createdb_path = "/usr/lib/postgresql/${::postgres_default_version}/bin/createdb"
|
|
$psql_path = "/usr/lib/postgresql/${::postgres_default_version}/bin/psql"
|
|
$datadir = "/var/lib/postgresql/${::postgres_default_version}/main"
|
|
$pg_hba_conf_path = "/etc/postgresql/${::postgres_default_version}/main/pg_hba.conf"
|
|
$postgresql_conf_path = "/etc/postgresql/${::postgres_default_version}/main/postgresql.conf"
|
|
$firewall_supported = false
|
|
$service_status = "/etc/init.d/${service_name} status | /bin/egrep -q 'Running clusters: .+'"
|
|
# TODO: not exactly sure yet what the right thing to do for Debian/Ubuntu is.
|
|
#$persist_firewall_command = '/sbin/iptables-save > /etc/iptables/rules.v4'
|
|
|
|
}
|
|
|
|
|
|
default: {
|
|
fail("Unsupported osfamily: ${::osfamily} operatingsystem: ${::operatingsystem}, module ${module_name} currently only supports osfamily RedHat and Debian")
|
|
}
|
|
}
|
|
|
|
}
|