From a9f4fe0639c2d501e3f1d429cc0978d8f752ab0b Mon Sep 17 00:00:00 2001
From: Nick Walker <nick.walker@puppetlabs.com>
Date: Tue, 29 Dec 2015 14:30:58 -0800
Subject: [PATCH] Add autovacuum settings to pe-postgresql

In the past I've seen the console and puppetdb databases grow
much larger than expected and I have a hypothesis that this is
because the default 20% setting for autovacuum is too high for our
workload.  So, there's a potential that changing this to 1%
might be too low and might reduce performance but it should
reduce disk usage and it can always be tuned upward if the
performance hit is too large, however, it is much more difficult
to reduce the size of a database after it has grown too large.

Basically, I'm erroring on the side of using less disk space and
hopefully less outages due to running out of disk space and we
can open a conversation on performance later if one needs to be had.
---
 .../manifests/pe_postgresql_management.pp     | 29 +++++++++++++++++++
 site/role/manifests/all_in_one_pe.pp          |  1 +
 2 files changed, 30 insertions(+)
 create mode 100644 site/profile/manifests/pe_postgresql_management.pp

diff --git a/site/profile/manifests/pe_postgresql_management.pp b/site/profile/manifests/pe_postgresql_management.pp
new file mode 100644
index 0000000..c4b81c3
--- /dev/null
+++ b/site/profile/manifests/pe_postgresql_management.pp
@@ -0,0 +1,29 @@
+class profile::pe_postgresql_management (
+  $autovacuum_scale_factor   = '.01',
+  $manage_postgresql_service = true,
+) {
+
+  $postgresql_service_resource_name = 'postgresqld'
+  $postgresql_service_name          = 'pe-postgresql'
+  $notify_postgresql_service        = $manage_postgresql_service ? {
+    true    => Service[$postgresql_service_resource_name],
+    default => undef,
+  }
+
+  if $manage_postgresql_service {
+    service { $postgresql_service_resource_name :
+      name   => $postgresql_service_name,
+      ensure => running,
+      enable => true,
+    }
+  }
+
+  #http://www.postgresql.org/docs/9.4/static/runtime-config-autovacuum.html
+  postgresql_conf { ['autovacuum_vacuum_scale_factor', 'autovacuum_analyze_scale_factor'] :
+    ensure => present,
+    target => '/opt/puppetlabs/server/data/postgresql/9.4/data/postgresql.conf',
+    value  => $autovacuum_scale_factor,
+    notify => $notify_postgresql_service,
+  }
+
+}
diff --git a/site/role/manifests/all_in_one_pe.pp b/site/role/manifests/all_in_one_pe.pp
index 9e93155..054ef28 100644
--- a/site/role/manifests/all_in_one_pe.pp
+++ b/site/role/manifests/all_in_one_pe.pp
@@ -2,5 +2,6 @@ class role::all_in_one_pe {
 
   include profile::puppetmaster
   include profile::git_webhook
+  include profile::pe_postgresql_management
 
 }