FIX

voxpupuli · Dec 14, 2023 · 9cb3fdc · 9cb3fdc
1 parent a52b5fa
commit 9cb3fdc
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/manifests/install/venv.pp b/manifests/install/venv.pp
@@ -3,7 +3,7 @@
   include python
 
   # Main python package bundle venv on some operating systems
-  unless $facts['os']['family'] in ['Archlinux', 'RedHat', 'FreeBSD'] {
+  unless $facts['os']['family'] in ['Archlinux', 'FreeBSD', 'RedHat'] {
     package { 'python-venv':
       ensure  => $python::venv,
       name    => "${python::install::python}-venv",

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -40,6 +40,8 @@
   }
   $manage_venv_package = $facts['os']['family'] ? {
     'Archlinux' => false,
+    'FreeBSD'   => false,
+    'RedHat'    => false,
     default     => true,
   }
 }
diff --git a/spec/classes/install/venv_spec.rb b/spec/classes/install/venv_spec.rb
@@ -26,7 +26,7 @@ class { 'python':
           PP
         end
 
-        if %w[Archlinux RedHat FreeBSD].include?(facts[:os]['family'])
+        if %w[Archlinux FreeBSD RedHat].include?(facts[:os]['family'])
           it { is_expected.not_to contain_package('python-venv') }
         else
           it { is_expected.to contain_package('python-venv').with(ensure: 'present') }