Update comments & remove backup param

Author: elainemccloskey

manifests/collect.pp

@@ -1,4 +1,9 @@
 # Create systemd units for repacking a given database type
+# @param database_type [String] The database to repack, uses titles from pg_repack.pp
+# @param command [String] defined in pg_repack.pp
+# @param disable_maintenance [Boolean] to disable maintenance mode (Default: false)
+# @param on_cal [String] values can be found in pg_repack.pp
+#
 define pe_databases::collect (
   String  $database_type       = $title,
   String  $command             = undef,

manifests/init.pp

@@ -1,10 +1,14 @@
-# Tuning, maintenance for PE PostgreSQL.
-#
-# @summary 
-#   Tuning, maintenance for PE PostgreSQL.
-
+# @summary Tuning, maintenance for PE PostgreSQL.
+# 
+# @param disable_maintenance [Boolean] true or false (Default: false)
+#   Disable or enable maintenance mode
+# @param manage_postgresql_settings [Boolean] true or false (Default: true)
+#   Manage PostgreSQL settings
+# @param manage_table_settings [Boolean] true or false (Default: false)
+#   Manage table settings
+# @param install_dir [String] Directory to install module into (Default: "/opt/puppetlabs/pe_databases")
+# @param scripts_dir [String] Directory to install scripts into (Default: "${install_dir}/scripts")
 class pe_databases (
-  Variant[Boolean,Undef] $manage_database_backups     = undef,
   # Manage the inclusion of the pg_repack class
   Boolean $manage_database_maintenance = true,
   # Manage the state of the maintenance tasks, i.e. systemd services and timers

manifests/pg_repack.pp

@@ -2,7 +2,10 @@
 #
 # @summary 
 #   Provides systemd timers to pg_repack tables in the pe-puppetdb database
-
+# 
+# @param disable_maintenance [Boolean] true or false (Default: false)
+#   Disable or enable maintenance mode
+# @param jobs [Integer] How many jobs to run in parallel
 class pe_databases::pg_repack (
   Boolean $disable_maintenance = false,
   Integer $jobs                = $facts['processors']['count'] / 4

manifests/postgresql_settings.pp

@@ -2,7 +2,12 @@
 #
 # @summary 
 #   Tune PostgreSQL settings 
-
+# 
+# @param maintenance_work_mem [String] Increase to improve speed of speed of vacuuming and reindexing (Example "1GB")
+# @param work_mem [String] Allows PostgreSQL to do larger in-memory sorts (Default: "4MB")
+# @param autovacumn_work_mem [String] Similar to but for maintenance_work_mem autovacuum processes only (Example "256MB")
+# @param autovacuum_max_workers [Integer] Maximum number of autovacuum processes to run concurrently (Default: 3)
+# 
 class pe_databases::postgresql_settings (
   # lint:ignore:140chars
   Float[0,1] $autovacuum_vacuum_scale_factor                    = 0.08,

manifests/postgresql_settings/table_settings.pp

@@ -2,7 +2,7 @@
 #
 # @summary 
 #   Tune PostgreSQL table settings
-
+# 
 class pe_databases::postgresql_settings::table_settings (
   # lint:ignore:140chars
   Boolean              $manage_reports_autovacuum_cost_delay     = lookup('pe_databases::postgresql_settings::manage_reports_autovacuum_cost_delay',    {'default_value' => true}),

manifests/set_puppetdb_table_autovacuum_cost_delay_zero.pp

@@ -2,7 +2,9 @@
 #
 # @summary 
 #   Defined type for PostgreSQL autovacuum_cost_delay
-
+# 
+# @param table_name [String]
+#   Name of the table
 define pe_databases::set_puppetdb_table_autovacuum_cost_delay_zero (
   String $table_name = $title,
 ) {

manifests/set_table_attribute.pp

@@ -2,7 +2,16 @@
 #
 # @summary 
 #   Defined type for PostgreSQL table attributes
-
+# 
+# @param db [String]
+#   Name of the database, this is pe-puppetdb for the uses of this module.
+# @param table_name [String] 
+#   Name of the table in the database.
+# @param table_attribute [String]
+#   Set to the table attribute value.
+# @param table_attribute_value [String]
+#   Value of setting for the table set in table_settings.pp
+#
 define pe_databases::set_table_attribute (
   String $db,
   String $table_name,