Merge pull request #39 from lbetz/add-type-retetion-options

Add new data type for retention options

REFERENCE.md

@@ -26,6 +26,8 @@ Configures IcingaDB
 
 ### Data types
 
+* [`IcingaDB::RetentionOptions`](#IcingaDB--RetentionOptions): Data type for retention options of the connection to
+the IcingaDB database.
 * [`IcingaDB::DBOptions`](#IcingaDB--DBOptions): Data type for options to connect the IcingaDB database.
 
 ## Classes
@@ -399,14 +401,7 @@ Default value: `undef`
 
 ##### <a name="-icingadb--retention_options"></a>`retention_options`
 
-Data type:
-
-```puppet
-Hash[Enum[
-      'acknowledgement','comment','downtime',
-      'flapping','notification','state'
-  ], Integer[1]]
-```
+Data type: `IcingaDB::RetentionOptions`
 
 Map of history category to number of days to retain its data in order to
 enable retention only for specific categories or to override the number
@@ -755,6 +750,11 @@ Data type: `Stdlib::Absolutepath`
 
 ## Data types
 
+### <a name="IcingaDB--RetentionOptions"></a>`IcingaDB::RetentionOptions`
+
+Data type for retention options of the connection to
+the IcingaDB database.
+
 ### <a name="IcingaDB--DBOptions"></a>`IcingaDB::DBOptions`
 
 Data type for options to connect the IcingaDB database.
@@ -763,11 +763,8 @@ Alias of
 
 ```puppet
 Hash[Enum[
-    'max_connections',
-    'max_connections_per_table',
-    'max_placeholders_per_statement',
-    'max_rows_per_transaction',
-    'wsrep_sync_wait'
+    'acknowledgement','comment','downtime',
+    'flapping','notification','state'
   ], Integer[1]]
 ```
 

manifests/init.pp

@@ -172,10 +172,7 @@
   Pattern[/^\d+\.?\d*[d|h|m|s]?$/]               $logging_interval       = '20s',
   Optional[Integer[1]]                           $retention_history_data = undef,
   Optional[Integer[1]]                           $retention_sla_data     = undef,
-  Hash[Enum[
-      'acknowledgement','comment','downtime',
-      'flapping','notification','state'
-  ], Integer[1]]                                 $retention_options      = {},
+  IcingaDB::RetentionOptions                     $retention_options      = {},
 ) {
   require icingadb::globals
 

types/retentionoptions.pp

@@ -0,0 +1,10 @@
+# @summary
+#   Data type for retention options of the connection to
+#   the IcingaDB database.
+type IcingaDB::RetentionOptions = Hash[
+  Enum[
+    'acknowledgement','comment','downtime',
+    'flapping','notification','state'
+  ],
+  Integer[1]
+]