Simplify variable use, add fact support

3 files changed, 15 insertions, 15 deletions

diff --git a/personal_infra/puppet/modules/dns_dhcp/manifests/init.pp b/personal_infra/puppet/modules/dns_dhcp/manifests/init.pp
index c7a35520..358633af 100644
--- a/personal_infra/puppet/modules/dns_dhcp/manifests/init.pp
+++ b/personal_infra/puppet/modules/dns_dhcp/manifests/init.pp
@@ -1,11 +1,11 @@
 class dns_dhcp {
-  $domain = lookup("'$ansible_inventory_hostname'.network.dns_dhcp.domain")
+  $domain = lookup("network.dns_dhcp.domain")
 
   package {'dnsmasq':}
   ->
   file {'/etc/dnsmasq.d/internal':
     content => epp('dns_dhcp/internal', {
-      'dns_dhcp' => lookup("'$ansible_inventory_hostname'.network.dns_dhcp"),
+      'dns_dhcp' => lookup("network.dns_dhcp"),
     }),
   }
   ~>
diff --git a/personal_infra/puppet/modules/proxmox/manifests/init.pp b/personal_infra/puppet/modules/proxmox/manifests/init.pp
index 21360368..0fa82d18 100644
--- a/personal_infra/puppet/modules/proxmox/manifests/init.pp
+++ b/personal_infra/puppet/modules/proxmox/manifests/init.pp
@@ -1,7 +1,7 @@
 class proxmox {
   file {'/etc/network/interfaces':
     content => epp('proxmox/interfaces', {
-      "network" => lookup("'$ansible_inventory_hostname'.network"),
+      "network" => lookup("network"),
     }),
   }
   ~>
diff --git a/personal_infra/puppet/site/00-common.pp b/personal_infra/puppet/site/00-common.pp
index c5023081..b542eb9e 100644
--- a/personal_infra/puppet/site/00-common.pp
+++ b/personal_infra/puppet/site/00-common.pp
@@ -1,28 +1,28 @@
 include automatic_updates
 
-$tinc_hosts = lookup("'$ansible_inventory_hostname'.groups.tinc")
-$tinc_other_hosts = $tinc_hosts.filter |$host_name| { $host_name != $ansible_inventory_hostname }
+$tinc_hosts = lookup("groups.tinc")
+$tinc_other_hosts = $tinc_hosts.filter |$host_name| { $host_name != $facts["networking"]["fqdn"] }
 
 $tinc_locations = Hash($tinc_hosts.map |$host_name| { [
-  lookup("'$host_name'.network.tinc.location"),
+  lookup("hostvars.'$host_name'.network.tinc.location"),
   {
-    subnet => lookup("'$host_name'.network.self_internal_network"),
-    address => lookup("'$host_name'.network.public_hostname"),
+    subnet => lookup("hostvars.'$host_name'.network.self_internal_network"),
+    address => lookup("hostvars.'$host_name'.network.public_hostname"),
   }
 ] })
 
-$tinc_connect_to = $tinc_other_hosts.map |$host_name| { lookup("'$host_name'.network.tinc.location") }
+$tinc_connect_to = $tinc_other_hosts.map |$host_name| { lookup("hostvars.'$host_name'.network.tinc.location") }
 
-$tinc_other_networks = $tinc_other_hosts.map |$host_name| { lookup("'$host_name'.network.self_internal_network") }
+$tinc_other_networks = $tinc_other_hosts.map |$host_name| { lookup("hostvars.'$host_name'.network.self_internal_network") }
 
-if 'tinc' in lookup("'$ansible_inventory_hostname'.group_names") {
+if 'tinc' in lookup("group_names") {
   class {'tinc':
-    tinc_name           => lookup("'$ansible_inventory_hostname'.tinc_global.name"),
-    tinc_location       => lookup("'$ansible_inventory_hostname'.network.tinc.location"),
+    tinc_name           => lookup("tinc_global.name"),
+    tinc_location       => lookup("network.tinc.location"),
     tinc_connect_to     => $tinc_connect_to,
     tinc_locations      => $tinc_locations,
-    tinc_ip             => lookup("'$ansible_inventory_hostname'.network.self_internal_ip"),
-    tinc_netmask        => lookup("'$ansible_inventory_hostname'.network.self_internal_netmask"),
+    tinc_ip             => lookup("network.self_internal_ip"),
+    tinc_netmask        => lookup("network.self_internal_netmask"),
     tinc_other_networks => $tinc_other_networks,
   }
 }