Merge branch 'develop' into neoloc/prometheus

site/profiles/manifests/ntp/client.pp

@@ -2,6 +2,10 @@
 # use exported resources from profiles::ntp::server if they are available
 class profiles::ntp::client (
   Array     $peers,
+  Variant[
+    String,
+    Undef
+  ]         $ntp_role = undef,
   Boolean   $wait_enable = true,
   Enum[
     'running',
@@ -14,16 +18,23 @@ class profiles::ntp::client (
   # through the profiles::ntp::server class.
   if $client_only {
 
+    # if ntp_role is set, find all hosts matching that enc_role
+    if $ntp_role == undef {
+      $ntpserver_array = $peers
+    }else{
+      $ntpserver_array = query_nodes("enc_role='${ntp_role}'", 'networking.fqdn')
+    }
+
     # Define the client configuration based on OS family
     if $facts['os']['family'] == 'RedHat' {
       class { 'chrony':
-        servers     => $peers,
+        servers     => $ntpserver_array,
         wait_enable => $wait_enable,
         wait_ensure => $wait_ensure,
       }
     } else {
       class { 'chrony':
-        servers    => $peers,
+        servers    => $ntpserver_array,
       }
     }
   }

site/profiles/manifests/ntp/server.pp

@@ -19,7 +19,7 @@ class profiles::ntp::server (
 
   # check the enc_role has been set, it can take two puppet runs to do this
   # TODO: change away from external fact
-  if $facts['enc_role'] == 'roles::infra::ntpserver' {
+  if $facts['enc_role'] == 'roles::infra::ntp::server' {
 
     # define the server
     if $facts['os']['family'] == 'RedHat' {

site/roles/manifests/infra/ntp/server.pp

@@ -1,5 +1,5 @@
 # a role to deploy a ntp server
-class roles::infra::ntpserver {
+class roles::infra::ntp::server {
   include profiles::defaults
   include profiles::base
   include profiles::ntp::server

site/roles/manifests/infra/puppet/master.pp

@@ -1,6 +1,6 @@
 # a role to deploy the puppetmaster
 # work in progress
-class roles::puppet::puppetmaster {
+class roles::infra::puppet::master {
     include profiles::defaults
     include profiles::base
     include profiles::puppet::puppetmaster

site/roles/manifests/infra/puppetboard/server.pp

@@ -1,5 +1,5 @@
 # a role to deploy the puppetboard
-class roles::puppet::puppetboard {
+class roles::infra::puppetboard::server {
     include profiles::defaults
     include profiles::base
     include profiles::puppet::puppetboard

site/roles/manifests/infra/puppetdb/api.pp

@@ -1,5 +1,5 @@
 # a role to deploy the puppetdb api service
-class roles::puppet::puppetdb_api {
+class roles::infra::puppetdb::api {
     include profiles::defaults
     include profiles::base
     include profiles::puppet::puppetdb_api

site/roles/manifests/infra/puppetdb/sql.pp

@@ -1,5 +1,5 @@
 # a role to deploy the puppetdb postgresql service
-class roles::puppet::puppetdb_sql {
+class roles::infra::puppetdb::sql {
     include profiles::defaults
     include profiles::base
     include profiles::puppet::puppetdb_sql

site/roles/manifests/infra/reposync/syncer.pp

@@ -1,5 +1,5 @@
 # a role to deploy a packagerepo
-class roles::infra::packagerepo {
+class roles::infra::reposync::syncer {
   include profiles::defaults
   include profiles::base
   include profiles::base::datavol

site/roles/manifests/puppet/puppetdb.pp

@@ -1,7 +0,0 @@
-# a role to deploy the puppetdb
-# work in progress
-class roles::puppet::puppetdb {
-    include profiles::defaults
-    include profiles::base
-    include profiles::puppet::puppetdb
-  }