From a05f81799d0e4040f6a76d6edfc926aa0e37c14e Mon Sep 17 00:00:00 2001
From: Ben Vincent <ben@unkin.net>
Date: Sat, 11 May 2024 14:36:28 +1000
Subject: [PATCH] fix: export cobbler DNS if is_cobbler_master

- set prodinf01n48 as primary cobbler node
- ensure the cobbler DNS record is created
---
 .../nodes/prodinf01n48.main.unkin.net.yaml     |  2 ++
 site/profiles/manifests/cobbler/config.pp      | 18 ++++++++++--------
 site/profiles/manifests/cobbler/params.pp      |  1 +
 3 files changed, 13 insertions(+), 8 deletions(-)
 create mode 100644 hieradata/nodes/prodinf01n48.main.unkin.net.yaml

diff --git a/hieradata/nodes/prodinf01n48.main.unkin.net.yaml b/hieradata/nodes/prodinf01n48.main.unkin.net.yaml
new file mode 100644
index 0000000..f7ad64b
--- /dev/null
+++ b/hieradata/nodes/prodinf01n48.main.unkin.net.yaml
@@ -0,0 +1,2 @@
+---
+profiles::cobbler::params::is_cobbler_master: true
diff --git a/site/profiles/manifests/cobbler/config.pp b/site/profiles/manifests/cobbler/config.pp
index 9b5c2af..90aee5d 100644
--- a/site/profiles/manifests/cobbler/config.pp
+++ b/site/profiles/manifests/cobbler/config.pp
@@ -7,6 +7,7 @@ class profiles::cobbler::config {
   $httpd_ssl_certificate = $profiles::cobbler::params::httpd_ssl_certificate
   $httpd_ssl_privatekey = $profiles::cobbler::params::httpd_ssl_privatekey
   $pxe_just_once = $profiles::cobbler::params::pxe_just_once
+  $is_cobbler_master = $profiles::cobbler::params::is_cobbler_master
   $service_cname = $profiles::cobbler::params::service_cname
   $next_server = $profiles::cobbler::params::next_server
   $server = $profiles::cobbler::params::server
@@ -64,12 +65,13 @@ class profiles::cobbler::config {
   }
 
   # export cnames for cobbler
-  #profiles::dns::record { "${::facts['networking']['fqdn']}_${service_cname}_CNAME":
-  #  value  => $::facts['networking']['hostname'],
-  #  type   => 'CNAME',
-  #  record => "${service_cname}.",
-  #  zone   => $::facts['networking']['domain'],
-  #  order  => 10,
-  #}
-
+  if $is_cobbler_master {
+    profiles::dns::record { "${::facts['networking']['fqdn']}_${service_cname}_CNAME":
+      value  => $::facts['networking']['hostname'],
+      type   => 'CNAME',
+      record => "${service_cname}.",
+      zone   => $::facts['networking']['domain'],
+      order  => 10,
+    }
+  }
 }
diff --git a/site/profiles/manifests/cobbler/params.pp b/site/profiles/manifests/cobbler/params.pp
index ca5ddfd..877f986 100644
--- a/site/profiles/manifests/cobbler/params.pp
+++ b/site/profiles/manifests/cobbler/params.pp
@@ -8,6 +8,7 @@ class profiles::cobbler::params (
   String  $server        = $::facts['networking']['ip'],
   String  $next_server   = $::facts['networking']['ip'],
   Boolean $pxe_just_once = true,
+  Boolean $is_cobbler_master = false,
   Array   $packages = [
       'cobbler',
       'cobbler3.2-web',