Eliminate params.pp

params.pp and Inheritance in general is not best Practice anymore.
Hiera provides better Definition of Parameters.

- move Defaultvalues to Modulelevel-Hiera
- eliminate params.pp
- specify Lookup-Options
- sort YAML-Files alphabetically

Attention: Parameter postfix::master_os_template still is without a Defaultvalue!

Author: cocker-cc

REFERENCE.md

@@ -1,6 +1,86 @@
 ---
-postfix::params::aliasesseltype: ~
-postfix::params::seltype: ~
-postfix::params::mailx_package: 'mailx'
-postfix::params::restart_cmd: '/etc/init.d/postfix reload'
-...
+lookup_options:
+  postfix::conffiles:
+    merge: deep
+  postfix::configs:
+    merge: deep
+  postfix::hashes:
+    merge: deep
+  postfix::ldap_packages:
+    merge: unique
+  postfix::mailaliases:
+    merge: deep
+  postfix::maps:
+    merge: deep
+  postfix::masquerade_classes:
+    merge: unique
+  postfix::masquerade_domains:
+    merge: unique
+  postfix::masquerade_exceptions:
+    merge: unique
+  postfix::master_entries:
+    merge: unique
+  postfix::mta::masquerade_classes:
+    merge: unique
+  postfix::mta::masquerade_domains:
+    merge: unique
+  postfix::mta::masquerade_exceptions:
+    merge: unique
+  postfix::root_mail_recipient:
+    merge: first
+  postfix::satellitemasquerade_classes:
+    merge: unique
+  postfix::satellitemasquerade_domains:
+    merge: unique
+  postfix::satellitemasquerade_exceptions:
+    merge: unique
+  postfix::smtp_listen:
+    merge: unique
+  postfix::transports:
+    merge: deep
+  postfix::virtuals:
+    merge: deep
+postfix::alias_maps: hash:/etc/aliases
+postfix::amavis_procs: 2
+postfix::confdir: "/etc/postfix"
+postfix::conffiles: {}
+postfix::configs: {}
+postfix::hashes: {}
+postfix::inet_interfaces: all
+postfix::inet_protocols: all
+postfix::ldap: false
+postfix::ldap_packages: []
+postfix::lookup_table_type: hash
+postfix::mail_user: vmail
+postfix::mailaliases: {}
+postfix::mailman: false
+postfix::mailx_ensure: present
+postfix::mailx_package: mailx
+postfix::maincf_source: puppet:///modules/postfix/main.cf
+postfix::manage_aliases: true
+postfix::manage_conffiles: true
+postfix::manage_mailname: true
+postfix::manage_mailx: true
+postfix::manage_root_alias: true
+postfix::maps: {}
+postfix::master_bounce_command: bounce
+postfix::master_defer_command: bounce
+postfix::master_entries: []
+postfix::mta: false
+postfix::mydestination: "$myhostname, localhost.$mydomain, localhost"
+postfix::mynetworks: 127.0.0.0/8 [::ffff:127.0.0.0]/104 [::1]/128
+postfix::myorigin: "%{facts.networking.fqdn}"
+postfix::postfix_ensure: present
+postfix::restart_cmd: "/etc/init.d/postfix reload"
+postfix::root_group: root
+postfix::root_mail_recipient: nobody
+postfix::satellite: false
+postfix::service_enabled: true
+postfix::service_ensure: running
+postfix::smtp_listen: 127.0.0.1
+postfix::transports: {}
+postfix::use_amavisd: false
+postfix::use_dovecot_lda: false
+postfix::use_schleuder: false
+postfix::use_sympa: false
+postfix::virtuals: {}

data/common.yaml

@@ -1,2 +1,2 @@
 ---
-postfix::params::master_os_template: "postfix/master.cf.debian.erb"
+postfix::master_os_template: postfix/master.cf.debian.erb

data/os/Alpine.yaml

@@ -2,7 +2,6 @@
 postfix::confdir: "/usr/local/etc/postfix"
 postfix::manage_mailname: false
 postfix::manage_mailx: false
-postfix::root_group: "wheel"
-postfix::params::master_os_template: "postfix/master.cf.FreeBSD.erb"
-postfix::params::restart_cmd: "/usr/local/etc/rc.d/postfix reload"
-...
+postfix::master_os_template: postfix/master.cf.FreeBSD.erb
+postfix::restart_cmd: "/usr/local/etc/rc.d/postfix reload"
+postfix::root_group: wheel

data/os/FreeBSD.yaml

@@ -1,5 +1,4 @@
 ---
-postfix::params::master_os_template: "postfix/master.cf.Solaris.erb"
-postfix::params::restart_cmd: '/usr/sbin/svcadm refresh network/smtp:postfix'
-postfix::params::mailx_package: 'mailx'
-...
+postfix::mailx_package: mailx
+postfix::master_os_template: postfix/master.cf.Solaris.erb
+postfix::restart_cmd: "/usr/sbin/svcadm refresh network/smtp:postfix"

data/os/Solaris.yaml

@@ -1,5 +1,5 @@
 ---
-postfix::params::mailx_package: 'bsd-mailx'
-postfix::params::master_os_template: 'postfix/master.cf.debian.erb'
-postfix::ldap_packages: ['postfix-ldap']
-...
+postfix::ldap_packages:
+- postfix-ldap
+postfix::mailx_package: bsd-mailx
+postfix::master_os_template: postfix/master.cf.debian.erb

data/osfamily/Debian.yaml

@@ -1,3 +1,2 @@
 ---
-postfix::params::mailx_package: 'mailx'
-...
+postfix::mailx_package: mailx

data/osfamily/Debian/etch.yaml

@@ -1,3 +1,2 @@
 ---
-postfix::params::mailx_package: 'mailx'
-...
+postfix::mailx_package: mailx

data/osfamily/Debian/lenny.yaml

@@ -1,3 +1,2 @@
 ---
-postfix::params::mailx_package: 'mailx'
-...
+postfix::mailx_package: mailx

data/osfamily/Debian/sarge.yaml

@@ -1,7 +1,6 @@
 ---
-postfix::params::aliasesseltype: 'etc_aliases_t'
-postfix::params::seltype: 'postfix_etc_t'
-postfix::params::restart_cmd: '/bin/systemctl reload postfix'
-postfix::params::master_os_template: 'postfix/master.cf.redhat.erb'
-postfix::mta_bin_path: '/usr/sbin/sendmail.postfix'
-...
+postfix::aliasesseltype: etc_aliases_t
+postfix::master_os_template: postfix/master.cf.redhat.erb
+postfix::mta_bin_path: "/usr/sbin/sendmail.postfix"
+postfix::restart_cmd: "/bin/systemctl reload postfix"
+postfix::seltype: postfix_etc_t