Changes RedHat repo to use the EPEL module

* Yum::Install to install the RPM is a bit more flakey 
* EPEL module is pretty ubiquitous, not that big of a deal to add it

.fixtures.yml

@@ -7,8 +7,7 @@ fixtures:
     concat:
       repo: 'git://github.com/puppetlabs/puppetlabs-concat'
       ref: '2.1.0'
-    yum:
-      repo: 'https://github.com/voxpupuli/puppet-yum.git'
-      ref: 'v0.10.0'
+    epel:
+      repo: 'https://github.com/stahnma/puppet-module-epel'
   symlinks:
     'collectd': "#{source_dir}"

manifests/plugin/cuda.pp

@@ -32,9 +32,9 @@
       if $::osfamily == 'RedHat' {
         # Epel is installed in install.pp if manage_repo is true
         # python-pip doesn't exist in base for RedHat. Need epel installed first
-        if (defined(Yum::Install['epel-release'])) {
+        if (defined(Class['::epel'])) {
           Package['python-pip'] {
-            require => Yum::Install['epel-release'],
+            require => Class['::epel'],
           }
         }
       }

manifests/plugin/rabbitmq.pp

@@ -87,9 +87,9 @@
       if $::osfamily == 'RedHat' {
         # Epel is installed in install.pp if manage_repo is true
         # python-pip doesn't exist in base for RedHat. Need epel installed first
-        if (defined(Yum::Install['epel-release'])) {
+        if (defined(Class['::epel'])) {
           Package['python-pip'] {
-            require => Yum::Install['epel-release'],
+            require => Class['::epel'],
           }
         }
       }

manifests/repo.pp

@@ -10,7 +10,7 @@
     $osfamily_downcase = downcase($::osfamily)
 
     if defined("::collectd::repo::${osfamily_downcase}") {
-      include "::collectd::repo::${osfamily_downcase}"
+      require "::collectd::repo::${osfamily_downcase}"
     } else {
       notify{"You have asked to manage_repo on a system that doesn't have a repo class specified: ${::osfamily}":}
     }

manifests/repo/redhat.pp

@@ -10,18 +10,7 @@
     }
 
   } else {
-
-    # TODO: Replace this with EPEL module requirement in Major version bump
-
-    if !defined(Yum::Install['epel-release']) {
-      yum::install { 'epel-release':
-        ensure => 'present',
-        source => "https://dl.fedoraproject.org/pub/epel/epel-release-latest-${::operatingsystemmajrelease}.noarch.rpm",
-        before => Package[$::collectd::package_name],
-      }
-    }
-
-
+    require ::epel
   }
 
 }

metadata.json

@@ -105,8 +105,8 @@
       "version_requirement": ">= 1.2.5"
     },
     {
-      "name": "puppet-yum",
-      "version_requirement": ">= 0.9.15 < 1.0.0"
+      "name": "stahnma-epel",
+      "version_requirement": ">= 1.2.2"
     },
     {
       "name": "puppetlabs-apt",

spec/classes/collectd_init_spec.rb

@@ -29,7 +29,7 @@
           )
         end
         if facts[:osfamily] == 'RedHat'
-          it { is_expected.to contain_yum__install('epel-release') }
+          it { is_expected.to contain_class('epel') }
         end
       end
 
@@ -191,7 +191,7 @@
           context 'and ci_package_repo empty' do
             let(:params) { { manage_repo: true } }
             if facts[:osfamily] == 'RedHat'
-              it { is_expected.to contain_yum__install('epel-release') }
+              it { is_expected.to contain_class('epel') }
             end
           end
 

spec/spec_helper_acceptance.rb

@@ -17,7 +17,7 @@
       on host, puppet('module', 'install', 'puppetlabs-apt'), acceptable_exit_codes: [0]
       on host, puppet('module', 'install', 'puppetlabs-stdlib'), acceptable_exit_codes: [0]
       on host, puppet('module', 'install', 'puppetlabs-concat'), acceptable_exit_codes: [0]
-      on host, puppet('module', 'install', 'puppet-yum'), acceptable_exit_codes: [0]
+      on host, puppet('module', 'install', 'stahnma-epel'), acceptable_exit_codes: [0]
     end
   end
 end