Merge pull request #878 from AntagonistHQ/checksums

add data_checksums option to initdb

Author: tphoney

README.md

@@ -479,6 +479,12 @@ Default value: `undef`, which is effectively 'C'.
 
 **On Debian, you'll need to ensure that the 'locales-all' package is installed for full functionality of PostgreSQL.**
 
+##### `data_checksums`
+
+Optional boolean to turn on data checksums during `initdb`.
+
+Default value: `undef`, which is the same as `false`.
+
 ##### `timezone`
 
 Sets the default timezone of the postgresql server. The postgresql built-in default is taking the systems timezone information.

manifests/globals.pp

@@ -50,6 +50,7 @@
 
   $encoding                 = undef,
   $locale                   = undef,
+  $data_checksums           = undef,
   $timezone                 = undef,
 
   $manage_pg_hba_conf       = undef,

manifests/params.pp

@@ -11,6 +11,7 @@
   $ipv6acls                   = []
   $encoding                   = $postgresql::globals::encoding
   $locale                     = $postgresql::globals::locale
+  $data_checksums             = $postgresql::globals::data_checksums
   $timezone                   = $postgresql::globals::timezone
   $service_ensure             = 'running'
   $service_enable             = true

manifests/server.pp

@@ -48,6 +48,7 @@
 
   $encoding                   = $postgresql::params::encoding,
   $locale                     = $postgresql::params::locale,
+  $data_checksums             = $postgresql::params::data_checksums,
   $timezone                   = $postgresql::params::timezone,
 
   $manage_pg_hba_conf         = $postgresql::params::manage_pg_hba_conf,

manifests/server/initdb.pp

@@ -7,6 +7,7 @@
   $logdir         = $postgresql::server::logdir
   $encoding       = $postgresql::server::encoding
   $locale         = $postgresql::server::locale
+  $data_checksums = $postgresql::server::data_checksums
   $group          = $postgresql::server::group
   $user           = $postgresql::server::user
   $psql_path      = $postgresql::server::psql_path
@@ -82,11 +83,17 @@
       $require_before_initdb = [$datadir]
     }
 
-    $initdb_command = $locale ? {
+    $ic_locale = $locale ? {
       undef   => $ic_xlog,
       default => "${ic_xlog} --locale '${locale}'"
     }
 
+    $initdb_command = $data_checksums ? {
+      undef   => $ic_locale,
+      false   => $ic_locale,
+      default => "${ic_locale} --data-checksums"
+    }
+
     # This runs the initdb command, we use the existance of the PG_VERSION
     # file to ensure we don't keep running this command.
     exec { 'postgresql_initdb':