abstract the tinc host part into its own define

manifests/host.pp

@@ -0,0 +1,36 @@
+# a host for a certain network
+# title must be:
+#    hostname@network
+#
+define tinc::host(
+  $public_key,
+  $ensure      = present,
+  $port        = 655,
+  $compression = 10,
+) {
+  # if absent the net should
+  # clean it up by itself
+  if $ensure == 'present' {
+    validate_re($name,'.+@.+')
+    $sp_name = split($name,'@')
+    $fqdn_tinc = $sp_name[0]
+    $net = $sp_name[1]
+
+    include tinc
+    if $tinc::uses_systemd {
+      $service_name = "tincd@${net}"
+    } else {
+      $service_name = 'tinc'
+    }
+
+    file{"/etc/tinc/${net}/hosts/${fqdn_tinc}":
+      content => template('tinc/host.erb'),
+      # to be sure that we manage that net
+      require => File["/etc/tinc/${net}/hosts"],
+      notify  => Service[$service_name],
+      owner   => root,
+      group   => 0,
+      mode    => '0600';
+    }
+  }
+}

manifests/instance.pp

@@ -112,15 +112,15 @@ define tinc::instance(
         group   => 0,
         mode    => '0600';
     }
-    # export and collect all the keys of this net
+    # export this host and collect all the other hosts
-    @@file { "/etc/tinc/${name}/hosts/${fqdn_tinc}":
+    @@tinc::host{"${fqdn_tinc}@${name}":
-      content => template('tinc/host.erb'),
+      port        => $port,
-      tag     => "tinc_host_for_${name}",
+      compression => $compression,
-      owner   => root,
+      address     => $host_address,
-      group   => 0,
+      public_key  => $tinc_keys[1],
-      mode    => '0600';
+      tag         => "tinc::host_for_${name}",
     }
-    File<<| tag == "tinc_host_for_${name}" |>>
+    Tinc::Host<<| tag == "tinc::host_for_${name}" |>>
 
     concat::fragment{"tinc_conf_header_${name}":
       target  => $tinc_config,

templates/host.erb

@@ -1,6 +1,5 @@
-Address = <%= @host_address %>
+Address = <%= @address %>
 Port    = <%= @port %>
 Compression = <%= @compression %> # 0=no,11=best lzo
-#
 
-<%= @tinc_keys[1] %>
+<%= @public_key %>