FIX

voxpupuli · Oct 9, 2024 · 04c346a · 04c346a
1 parent 8491f8e
commit 04c346a
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 7 deletions.
diff --git a/REFERENCE.md b/REFERENCE.md
@@ -86,6 +86,7 @@ The following parameters are available in the `python` class:
 * [`manage_venv_package`](#-python--manage_venv_package)
 * [`manage_pip_package`](#-python--manage_pip_package)
 * [`venv`](#-python--venv)
+* [`pip_package_name`](#-python--pip_package_name)
 * [`gunicorn_package_name`](#-python--gunicorn_package_name)
 * [`python_pips`](#-python--python_pips)
 * [`python_pyvenvs`](#-python--python_pyvenvs)
@@ -228,6 +229,14 @@ Data type: `Python::Package::Ensure`
 
 Default value: `'absent'`
 
+##### <a name="-python--pip_package_name"></a>`pip_package_name`
+
+Data type: `Optional[String[1]]`
+
+
+
+Default value: `undef`
+
 ##### <a name="-python--gunicorn_package_name"></a>`gunicorn_package_name`
 
 Data type: `String[1]`

diff --git a/manifests/init.pp b/manifests/init.pp
@@ -76,7 +76,6 @@
     }
   }
 
-
   contain python::install
   contain python::config
 

diff --git a/manifests/install/pip.pp b/manifests/install/pip.pp
@@ -15,7 +15,7 @@
     }
     'anaconda': {
     }
-    pip, default: {
+    'pip', default: {
       case $facts['os']['family'] {
         'AIX': {
           unless String($python::version) =~ /^python3/ {

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') }

diff --git a/spec/classes/install_pip_spec.rb b/spec/classes/install_pip_spec.rb
@@ -16,9 +16,9 @@
       context 'when ensuring pip is absent' do
         let(:pre_condition) do
           <<~PP
-          class { 'python':
-            pip => absent,
-          }
+            class { 'python':
+              pip => absent,
+            }
           PP
         end
-Original file line number
+Diff line change
@@ Expand Up / @@ -76,7 +76,6 @@ @@
         }
       }
       contain python::install
       contain python::config
@@ Expand Down @@