Don't break OS python installation

voxpupuli#879
kcl-nmssys · Mar 2, 2020 · ac627bc · ac627bc
1 parent 35b7811
commit ac627bc
Showing 1 changed file with 13 additions and 11 deletions.
diff --git a/manifests/plugin/python.pp b/manifests/plugin/python.pp
@@ -52,17 +52,19 @@
     default  => 'directory',
   }
 
-  ensure_resource('file', $module_dirs,
-    {
-      'ensure'  => $ensure_modulepath,
-      'mode'    => $collectd::plugin_conf_dir_mode,
-      'owner'   => $collectd::config_owner,
-      'purge'   => $collectd::purge_config,
-      'force'   => true,
-      'group'   => $collectd::config_group,
-      'require' => Package[$collectd::package_name]
-    }
-  )
+  if $module_dirs == $modulepaths {
+    ensure_resource('file', $module_dirs,
+      {
+        'ensure'  => $ensure_modulepath,
+        'mode'    => $collectd::plugin_conf_dir_mode,
+        'owner'   => $collectd::config_owner,
+        'purge'   => $collectd::purge_config,
+        'force'   => true,
+        'group'   => $collectd::config_group,
+        'require' => Package[$collectd::package_name]
+      }
+    )
+  }
 
   # should be loaded after global plugin configuration
   $python_conf = "${collectd::plugin_conf_dir}/${conf_name}"