search

saz / puppet-sudo

Manage sudo with Puppet on Debian-, RedHat- and SUSE-based linux distributions and some BSDs
Other
107 stars 215 forks source link

rpm installed override sudoversion package #182

Closed Rocco83 closed 7 years ago

Rocco83 commented 7 years ago

The sudoversion is populated in first instance by rpm provider. If rpm command is found, other check are not performed.

This is not correct in example in debian with rpm exec installed, because it will reply in facts: "package sudo is not installed"

way out suggested: a) check osfamily before, and perform rpm command only if this is osfamily redhat, ... b) check the fact after setting the variable, if the value is not correct move to exec sudo.

Second is more robust in case of package manually installed.

Rocco83 commented 7 years ago

Ok sorry, i was checking an old version. This is already fixed.