Merge pull request #740 from StatensPensjonskasse/real-yaml-in-mongod…

….conf Use proper yaml for the mongod config
voxpupuli · Apr 15, 2024 · daaa242 · daaa242
2 parents 4820f27 + 14f6fc9
commit daaa242
Show file tree

Hide file tree

Showing 4 changed files with 441 additions and 287 deletions.
diff --git a/REFERENCE.md b/REFERENCE.md
@@ -1084,11 +1084,11 @@ Default value: `undef`
 
 ##### <a name="-mongodb--server--fork"></a>`fork`
 
-Data type: `Boolean`
+Data type: `Optional[Boolean]`
 
 Set to true to fork server process at launch time. The default setting depends on the operating system.
 
-Default value: `false`
+Default value: `undef`
 
 ##### <a name="-mongodb--server--port"></a>`port`
 
@@ -1393,9 +1393,13 @@ Default value: `undef`
 
 ##### <a name="-mongodb--server--set_parameter"></a>`set_parameter`
 
-Data type: `Optional[Variant[String[1], Array[String[1]]]]`
+Data type: `Optional[Variant[String[1], Array[String[1]], Hash]]`
 
-Specify extra configuration file parameters (i.e. textSearchEnabled=true).
+Set MongoDB parameters
+Supported types:
+  String (i.e. 'textSearchEnabled=true' or 'textSearchEnabled: true' )
+  Array  (i.e. ['textSearchEnabled=true'] or ['textSearchEnabled: true'] )
+  Hash   (i.e. {'textSearchEnabled' => true}
 
 Default value: `undef`
 
@@ -1427,7 +1431,8 @@ Default value: `undef`
 
 Data type: `Optional[Hash]`
 
-A hash to allow for additional configuration options to be set in user-provided template.
+A hash to allow for additional configuration options to be set.
+(i.e {'security' => { 'javascriptEnabled' => false}})
 
 Default value: `undef`
 
@@ -1457,19 +1462,19 @@ Default value: `undef`
 
 ##### <a name="-mongodb--server--tls_conn_without_cert"></a>`tls_conn_without_cert`
 
-Data type: `Boolean`
+Data type: `Optional[Boolean]`
 
 Set to true to bypass client certificate validation for clients that do not present a certificate.
 
-Default value: `false`
+Default value: `undef`
 
 ##### <a name="-mongodb--server--tls_invalid_hostnames"></a>`tls_invalid_hostnames`
 
-Data type: `Boolean`
+Data type: `Optional[Boolean]`
 
 Set to true to disable the validation of the hostnames in TLS certificates.
 
-Default value: `false`
+Default value: `undef`
 
 ##### <a name="-mongodb--server--tls_mode"></a>`tls_mode`
 

diff --git a/manifests/server.pp b/manifests/server.pp
@@ -227,7 +227,11 @@
 #   replica set members.
 #
 # @param set_parameter
-#   Specify extra configuration file parameters (i.e. textSearchEnabled=true).
+#   Set MongoDB parameters
+#   Supported types:
+#     String (i.e. 'textSearchEnabled=true' or 'textSearchEnabled: true' )
+#     Array  (i.e. ['textSearchEnabled=true'] or ['textSearchEnabled: true'] )
+#     Hash   (i.e. {'textSearchEnabled' => true}
 #
 # @param syslog
 #   Sends all logging output to the host’s syslog system rather than to standard output or a log file.
@@ -239,7 +243,8 @@
 #   Path to the config template if the default doesn't match one needs.
 #
 # @param config_data
-#   A hash to allow for additional configuration options to be set in user-provided template.
+#   A hash to allow for additional configuration options to be set.
+#   (i.e {'security' => { 'javascriptEnabled' => false}})
 #
 # @param tls
 #   Ensure tls is enabled.
@@ -318,7 +323,7 @@
   Optional[Boolean] $ipv6                                                 = undef,
   Boolean $logappend                                                      = true,
   Optional[String] $system_logrotate                                      = undef,
-  Boolean $fork                                                           = false,
+  Optional[Boolean] $fork                                                 = undef,
   Optional[Integer[1, 65535]] $port                                       = undef,
   Optional[Boolean] $journal                                              = undef,
   Optional[Boolean] $smallfiles                                           = undef,
@@ -353,16 +358,16 @@
   Optional[Integer] $slowms                                               = undef,
   Optional[Stdlib::Absolutepath] $keyfile                                 = undef,
   Optional[Variant[String[6], Sensitive[String[6]]]] $key                 = undef,
-  Optional[Variant[String[1], Array[String[1]]]] $set_parameter           = undef,
+  Optional[Variant[String[1], Array[String[1]], Hash]] $set_parameter     = undef,
   Boolean $syslog                                                         = false,
   $config_content                                                         = undef,
   Optional[String] $config_template                                       = undef,
   Optional[Hash] $config_data                                             = undef,
   Boolean $tls                                                            = false,
   Optional[Stdlib::Absolutepath] $tls_key                                 = undef,
   Optional[Stdlib::Absolutepath] $tls_ca                                  = undef,
-  Boolean $tls_conn_without_cert                                          = false,
-  Boolean $tls_invalid_hostnames                                          = false,
+  Optional[Boolean] $tls_conn_without_cert                                = undef,
+  Optional[Boolean] $tls_invalid_hostnames                                = undef,
   Enum['requireTLS', 'preferTLS', 'allowTLS'] $tls_mode                   = 'requireTLS',
   Boolean $restart                                                        = true,
   Optional[String] $storage_engine                                        = undef,