From 087e44bb1aa01ce0ecebefe7b694c9b8331a6ef2 Mon Sep 17 00:00:00 2001
From: Silvio Rhatto <rhatto@riseup.net>
Date: Sun, 6 Sep 2020 20:48:17 -0300
Subject: Fix: hash merge on classes and resources lookups

---
 manifests/init.pp      |  2 +-
 manifests/resources.pp | 28 ++++++++++++++--------------
 2 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/manifests/init.pp b/manifests/init.pp
index 5015adc..f3f41a3 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -26,7 +26,7 @@ class nodo(
   }
 
   # Include additional classes
-  lookup('classes', { merge => unique, default_value => [] }).include
+  lookup('classes', { merge => hash, default_value => [] }).include
 
   # Declare resources
   class { "nodo::resources": }
diff --git a/manifests/resources.pp b/manifests/resources.pp
index 8fe188f..940e544 100644
--- a/manifests/resources.pp
+++ b/manifests/resources.pp
@@ -5,58 +5,58 @@ class nodo::resources {
   }
 
   # Packages
-  $packages = lookup('packages', undef, undef, {})
+  $packages = lookup('packages', { merge => hash, default_value => [] })
   create_resources('package', $packages)
 
   # Files
-  $files = lookup('files', undef, undef, {})
+  $files = lookup('files', { merge => hash, default_value => {} })
   create_resources('file', $files)
 
   # File lines
-  $file_lines = lookup('file_lines', undef, undef, {})
+  $file_lines = lookup('file_lines', { merge => hash, default_value => {} })
   create_resources('file_line', $file_lines)
 
   # Virtual machines
-  $vms = lookup('virtual::machines', undef, undef, {})
+  $vms = lookup('virtual::machines', { merge => hash, default_value => {} })
   create_resources('nodo::subsystem::virtual::instance', $vms)
 
   # Users
-  $users = lookup('users', undef, undef, {})
+  $users = lookup('users', { merge => hash, default_value => {} })
   create_resources('user::manage', $users)
 
   # Backup users
-  $backup_users = lookup('backup::users', undef, undef, {})
+  $backup_users = lookup('backup_users', { merge => hash, default_value => {} })
   create_resources('backup::user', $backup_users)
 
   # Cron entries
-  $cron = lookup('cron::jobs', undef, undef, {})
+  $cron = lookup('cron::jobs', { merge => hash, default_value => {} })
   create_resources('cron', $cron)
 
   # Hosts
-  $hosts = lookup('hosts', undef, undef, {})
+  $hosts = lookup('hosts', { merge => hash, default_value => {} })
   create_resources('host', $hosts)
 
   # Nginx entries
-  $nginx = lookup('nginx::sites', undef, undef, {})
+  $nginx = lookup('nginx::sites', { merge => hash, default_value => {} })
   create_resources('nginx::site', $nginx)
 
   # SSH keys
-  $sshkeys = lookup('sshkeys', undef, undef, {})
+  $sshkeys = lookup('sshkeys', { merge => hash, default_value => {} })
   create_resources('sshkey', $sshkeys)
 
   # SSH Authorized keys
-  $ssh_authorized_keys = lookup('ssh_authorized_keys', undef, undef, {})
+  $ssh_authorized_keys = lookup('ssh_authorized_keys', { merge => hash, default_value => {} })
   create_resources('ssh_authorized_key', $ssh_authorized_keys)
 
   # PaX flags
-  $pax = lookup('nodo::subsystem::grsec::paxctl', undef, undef, {})
+  $pax = lookup('nodo::subsystem::grsec::paxctl', { merge => hash, default_value => {} })
   create_resources('nodo::subsystem::grsec::paxctl', $pax)
 
   # Exec
-  $execs = lookup('execs', undef, undef, {})
+  $execs = lookup('execs', { merge => hash, default_value => {} })
   create_resources('exec', $execs)
 
   # Repositories
-  $vcsrepos = lookup('vcsrepos', undef, undef, {})
+  $vcsrepos = lookup('vcsrepos', { merge => hash, default_value => {} })
   create_resources('vcsrepo', $vcsrepos)
 }
-- 
cgit v1.2.3