Merge pull request #140 from treydock/latest-group

Support ensuring all yum group packages are installed
voxpupuli · Jan 30, 2020 · 7fc2d1f · 7fc2d1f
2 parents b527ee1 + 53dc1d7
commit 7fc2d1f
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/manifests/group.pp b/manifests/group.pp
@@ -20,7 +20,7 @@
 #
 define yum::group (
   Array[String[1]]                                    $install_options = [],
-  Enum['present', 'installed', 'absent', 'purged'] $ensure             = 'present',
+  Enum['present', 'installed', 'latest', 'absent', 'purged'] $ensure   = 'present',
   Optional[Integer] $timeout                                           = undef,
 ) {
 
@@ -36,6 +36,14 @@
         unless  => "yum grouplist hidden '${name}' | egrep -i '^Installed.+Groups:$'",
         timeout => $timeout,
       }
+      if $ensure == 'latest' {
+        exec { "yum-groupinstall-${name}-latest":
+          command => join(concat(["yum -y groupinstall '${name}'"], $install_options), ' '),
+          onlyif  => "yum groupinfo '${name}' | egrep '\\s+\\+'",
+          timeout => $timeout,
+          require => Exec["yum-groupinstall-${name}"],
+        }
+      }
     }
 
     'absent', 'purged': {

diff --git a/spec/defines/group_spec.rb b/spec/defines/group_spec.rb
@@ -30,4 +30,13 @@
     it { is_expected.to compile.with_all_deps }
     it { is_expected.to contain_exec("yum-groupinstall-#{title}").with_command("yum -y groupinstall 'Core' --enablerepo=epel") }
   end
+
+  context 'when ensure is set to `latest`' do
+    let(:title) { 'Core' }
+    let(:params) { { ensure: 'latest' } }
+
+    it { is_expected.to compile.with_all_deps }
+    it { is_expected.to contain_exec("yum-groupinstall-#{title}").with_command("yum -y groupinstall 'Core'") }
+    it { is_expected.to contain_exec("yum-groupinstall-#{title}-latest").with_command("yum -y groupinstall 'Core'") }
+  end
 end