Merge pull request #131 from m0dular/aph/system-warning

Change system notify to warning function

Author: jarretlavallee

manifests/system.pp

@@ -66,11 +66,9 @@
     contain puppet_metrics_collector::system::cpu
     contain puppet_metrics_collector::system::memory
     contain puppet_metrics_collector::system::processes
-  } else {
-    notify { 'sysstat_missing_warning':
-      message  => 'System collection disabled. Set `puppet_metrics_collector::system::manage_sysstat: true` to enable system metrics',
-      loglevel => warning,
-    }
+  }
+  else {
+    warning('System collection disabled. Set `puppet_metrics_collector::system::manage_sysstat: true` to enable system metrics')
   }
 
   if $facts['virtual'] == 'vmware' {

spec/classes/puppet_metrics_collector_system_spec.rb

@@ -11,7 +11,6 @@
       let(:pre_condition) { 'package{"sysstat": }' }
       let(:facts) { { puppet_metrics_collector: { have_sysstat: true, have_systemd: true } } }
 
-      it { is_expected.not_to contain_notify('sysstat_missing_warning') }
       it { is_expected.to contain_class('puppet_metrics_collector::system::cpu') }
       it { is_expected.to contain_class('puppet_metrics_collector::system::memory') }
       it { is_expected.to contain_class('puppet_metrics_collector::system::processes') }
@@ -21,15 +20,13 @@
       let(:params) { { manage_sysstat: true } }
       let(:facts) { { puppet_metrics_collector: { have_sysstat: false, have_systemd: true } } }
 
-      it { is_expected.not_to contain_notify('sysstat_missing_warning') }
       it { is_expected.to contain_package('sysstat') }
       it { is_expected.to contain_class('puppet_metrics_collector::system::cpu') }
       it { is_expected.to contain_class('puppet_metrics_collector::system::memory') }
       it { is_expected.to contain_class('puppet_metrics_collector::system::processes') }
     end
 
     context 'not installed and not managed' do
-      it { is_expected.to contain_notify('sysstat_missing_warning') }
       it { is_expected.not_to contain_package('sysstat') }
       it { is_expected.not_to contain_class('puppet_metrics_collector::system::cpu') }
       it { is_expected.not_to contain_class('puppet_metrics_collector::system::memory') }