search

saz / puppet-sudo

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

Updated lense with the standard lense /usr/share/augeas/lenses/dist/s… #129

Closed mukeshdevops closed 8 years ago

mukeshdevops commented 9 years ago

…udoers.aug shipped with augtool 1.2.0

Closes #128

fraenki commented 9 years ago

I'm wondering if the custom augeas lense is still required at all? I've read on 902b8ca6bc6cd00867216d1c42e9444278a1c3b4 that it was added to add proper support for RHEL 5 where the available augease lense is outdated/broken.

But as it currently stands this breaks augeas for other puppet-modules, unaware of the fixedsudoers lense:

Debug: Augeas[sudo-foreman-proxy](provider=augeas): /augeas/files/etc/sudoers.d/80_foreman-proxy/error = mxfm_load
Debug: Augeas[sudo-foreman-proxy](provider=augeas): /augeas/files/etc/sudoers.d/80_foreman-proxy/error/message = Lenses @FixedSudoers and @Sudoers could be used to load this file

Maybe the author of 902b8ca6bc6cd00867216d1c42e9444278a1c3b4 @tosmi could comment on this.

tosmi commented 9 years ago

i think that's related to #84. so the fix would be to remove the xfm line. i'll try to open a pull request for #84 within the next days.

saz commented 8 years ago

This PR should not be required due to the merge of #135