Add possibility to configure defined types via hiera

.travis.yml

@@ -27,7 +27,6 @@ matrix:
     env: PUPPET_GEM_VERSION="~> 4.6.0" STRICT_VARIABLES=yes
   - rvm: 1.9.3
     env: PUPPET_GEM_VERSION="~> 5.0.1" STRICT_VARIABLES=yes
-  # eclude all puppet 3.x 5.x for ruby 2.3
   - rvm: 2.3
     env: PUPPET_GEM_VERSION="~> 5.0.1" STRICT_VARIABLES=yes
   # eclude all puppet 3.x 4.x for ruby 2.4

README.md

@@ -15,6 +15,7 @@
   1. [Monitor (Tray)](#monitor)
   1. [Storage](#storage)
   1. [Web UI](#webui)
+  1. [Hiera](#hiera)
 1. [Limitations - OS compatibility, etc.](#limitations)
 
 ## Description
@@ -391,6 +392,31 @@ Additional configuration is required on the **Director** server.
 }
 ```
 
+### Hiera
+Since define ressource are used in the module they can not be used directly via hiera. Execptions are ofcourse the three classes mentioned in [Reference](#reference).
+But there exist hashes for the configuration directives in the `bareos::director` and the `bareos::webui` classes which can be used via hiera.
+
+```
+classes:
+  - bareos::director
+bareos::director::director::name_director: 'example'
+bareos::director::director::password: 'foobar'
+bareos::director::catalogs:
+  'testing':
+    db_driver: 'postgresql'
+    db_name: 'test'
+bareos::director::clients:
+  'alice':
+    address: foo.bar
+    password: foobar
+bareos::director::jobs:
+  'backup-alice':
+    messages: testing
+    pool: testing
+    type: backup
+    client: alice
+    file_set: testing
+
 ## Limitations
 
 This module is built upon and tested against the versions of Puppet listed in the metadata.json file (i.e. the listed compatible versions on the Puppet Forge).

manifests/director.pp

@@ -15,6 +15,19 @@
   $service_enable             = $::bareos::service_enable,
   $config_dir                 = "${::bareos::config_dir}/bareos-dir.d",
   Array[String] $managed_dirs = $::bareos::director_managed_dirs,
+  Hash $catalogs              = {},
+  Hash $clients               = {},
+  Hash $consoles              = {},
+  Hash $counters              = {},
+  Hash $directors             = {},
+  Hash $filesets              = {},
+  Hash $jobs                  = {},
+  Hash $jobdefs               = {},
+  Hash $messages              = {},
+  Hash $pools                 = {},
+  Hash $profiles              = {},
+  Hash $schedules             = {},
+  Hash $storages              = {},
 ) inherits ::bareos {
   include ::bareos::director::director
 
@@ -67,4 +80,60 @@
       tag         => ['bareos', 'bareos_director'],
     }
   }
+
+  $catalogs.each |String $resource, Hash $attributes| {
+    bareos::director::catalog { $resource:
+      * => $attributes;
+    }
+  }
+  $clients.each |String $resource, Hash $attributes| {
+    bareos::director::client { $resource:
+      * => $attributes;
+    }
+  }
+  $consoles.each |String $resource, Hash $attributes| {
+    bareos::director::console { $resource:
+      * => $attributes;
+    }
+  }
+  $filesets.each |String $resource, Hash $attributes| {
+    bareos::director::fileset { $resource:
+      * => $attributes;
+    }
+  }
+  $jobs.each |String $resource, Hash $attributes| {
+    bareos::director::job { $resource:
+      * => $attributes;
+    }
+  }
+  $jobdefs.each |String $resource, Hash $attributes| {
+    bareos::director::jobdefs { $resource:
+      * => $attributes;
+    }
+  }
+  $messages.each |String $resource, Hash $attributes| {
+    bareos::director::messages { $resource:
+      * => $attributes;
+    }
+  }
+  $pools.each |String $resource, Hash $attributes| {
+    bareos::director::pool { $resource:
+      * => $attributes;
+    }
+  }
+  $profiles.each |String $resource, Hash $attributes| {
+    bareos::director::profile { $resource:
+      * => $attributes;
+    }
+  }
+  $schedules.each |String $resource, Hash $attributes| {
+    bareos::director::schedule { $resource:
+      * => $attributes;
+    }
+  }
+  $storages.each |String $resource, Hash $attributes| {
+    bareos::director::storage { $resource:
+      * => $attributes;
+    }
+  }
 }

spec/classes/director_spec.rb

@@ -1,9 +1,26 @@
 require 'spec_helper'
 describe 'bareos::director' do
-  let(:pre_condition) { 'class {"::bareos::director::director": password => "password" }' }
-
   context 'with default values for all parameters' do
     it { is_expected.to compile }
     it { is_expected.to contain_class('bareos') }
   end
+  context 'with catalogs => { test: { db_driver: "sqlite", db_name: "test" }}}' do
+    let(:params) do
+      {
+        catalogs: {
+          test: {
+            db_driver: "sqlite",
+            db_name: "test",
+          }
+        }
+      }
+    end
+
+    it { is_expected.to compile }
+    it do
+      is_expected.to contain_bareos__director__catalog('test')
+        .with_db_driver('sqlite')
+        .with_db_name('test')
+    end
+  end
 end