(GH-cat-9) syntax:hiera:yaml fixes

Author: david22swan

examples/init.pp

@@ -1,11 +1,9 @@
 node default {
-
   notify { 'enduser-before': }
   notify { 'enduser-after': }
 
   class { 'ntp':
     require => Notify['enduser-before'],
     before  => Notify['enduser-after'],
   }
-
 }

manifests/config.pp

@@ -4,19 +4,16 @@
 # @api private
 #
 class ntp::config {
-
   #The servers-netconfig file overrides NTP config on SLES 12, interfering with our configuration.
   if ($facts['operatingsystem'] == 'SLES' and $facts['operatingsystemmajrelease'] == '12') or
-    ($facts['operatingsystem'] == 'OpenSuSE' and $facts['operatingsystemmajrelease'] == '42') {
+  ($facts['operatingsystem'] == 'OpenSuSE' and $facts['operatingsystemmajrelease'] == '42') {
     file { '/var/run/ntp/servers-netconfig':
-      ensure => 'absent'
+      ensure => 'absent',
     }
   }
 
-  case $::osfamily
-  {
-    'redhat':
-    {
+  case $::osfamily {
+    'redhat': {
       $daemon_config = '/etc/sysconfig/ntpd'
       if $ntp::daemon_extra_opts {
         file_line { 'Set NTPD daemon options':
@@ -35,8 +32,7 @@
         }
       }
     }
-    'Debian':
-    {
+    'Debian': {
       $daemon_config = '/etc/default/ntp'
       if $ntp::daemon_extra_opts {
         file_line { 'Set NTPD daemon options':
@@ -55,8 +51,7 @@
         }
       }
     }
-    'Suse':
-    {
+    'Suse': {
       $daemon_config = '/etc/sysconfig/ntp'
       if $ntp::daemon_extra_opts {
         file_line { 'Set NTPD daemon options':
@@ -67,7 +62,9 @@
         }
       }
     }
-    default: { }
+    default: {
+      # Empty
+    }
   }
 
   if $ntp::keys_enable {
@@ -123,7 +120,7 @@
       $step_ticker_content = template($ntp::step_tickers_template)
     } elsif $::ntp::step_tickers_epp {
       $step_ticker_content = epp($::ntp::step_tickers_epp)
-    } else{
+    } else {
       $step_ticker_content = epp('ntp/step-tickers.epp')
     }
 

manifests/install.pp

@@ -4,13 +4,9 @@
 # @api private
 #
 class ntp::install {
-
   if $ntp::package_manage {
-
     package { $ntp::package_name:
       ensure => $ntp::package_ensure,
     }
-
   }
-
 }

manifests/service.pp

@@ -4,7 +4,6 @@
 # @api private
 #
 class ntp::service {
-
   if $ntp::service_manage == true {
     service { 'ntp':
       ensure     => $ntp::service_ensure,
@@ -15,5 +14,4 @@
       hasrestart => $ntp::service_hasrestart,
     }
   }
-
 }

plans/acceptance/pe_server.pp

@@ -6,7 +6,7 @@
 #   ntp::acceptance::pe_server
 plan ntp::acceptance::pe_server(
   Optional[String] $version = '2019.8.5',
-  Optional[Hash] $pe_settings = {password => 'puppetlabs'}
+  Optional[Hash] $pe_settings = { password => 'puppetlabs' }
 ) {
   #identify pe server node
   $puppet_server =  get_targets('*').filter |$n| { $n.vars['role'] == 'ntpserver' }
@@ -19,4 +19,3 @@
     'pe_settings' => $pe_settings
   )
 }
-

plans/acceptance/provision_integration.pp

@@ -9,8 +9,6 @@
   Optional[String] $provision_type = 'provision_service',
 ) {
   #provision server machine, set role
-  run_task("provision::$provision_type", 'localhost',
-    action => 'provision', platform => $image, vars => 'role: ntpserver')
-  run_task("provision::$provision_type", 'localhost',
-    action => 'provision', platform => $image, vars => 'role: ntpclient')
+  run_task("provision::$provision_type", 'localhost', action => 'provision', platform => $image, vars => 'role: ntpserver')
+  run_task("provision::$provision_type", 'localhost', action => 'provision', platform => $image, vars => 'role: ntpclient')
 }