Fix $openldap::server::database::dboptions data type

PR #343 was right about this and the point was missed in #348 which did not fixed it. Fix it for good!
voxpupuli · Feb 1, 2022 · 39d3bfc · 39d3bfc
1 parent 0aaa376
commit 39d3bfc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/manifests/server/database.pp b/manifests/server/database.pp
@@ -15,7 +15,7 @@
   Optional[String[1]]                           $updateref       = undef,
   Array[String[1]]                              $limits          = [],
   # BDB/HDB options
-  Array[String[1]]                              $dboptions       = [],
+  Hash[String[1],Variant[String[1],Array[String[1]]]] $dboptions = {},
   Optional[String[1]]                           $synctype        = undef,
   # Synchronization options
   Optional[Boolean]                             $mirrormode      = undef,

diff --git a/spec/defines/openldap_server_database_spec.rb b/spec/defines/openldap_server_database_spec.rb
@@ -42,12 +42,14 @@
                   'dn.exact="cn=personnel,dc=example,dc=org" size=unlimited',
                   'dn.exact="cn=dirsync,dc=example,dc=org" size=100000'
                 ],
-                dboptions: [
-                  'set_cachesize 0 10485760 0',
-                  'set_lg_bsize 2097512',
-                  'set_lg_dir /var/tmp/bdb-log',
-                  'set_flags DB_LOG_AUTOREMOVE',
-                ],
+                dboptions: {
+                  config: [
+                    'set_cachesize 0 10485760 0',
+                    'set_lg_bsize 2097512',
+                    'set_lg_dir /var/tmp/bdb-log',
+                    'set_flags DB_LOG_AUTOREMOVE',
+                  ],
+                },
                 synctype: 'inclusive',
                 mirrormode: true,
                 syncusesubentry: 'wxw',