Closed rnelson0 closed 7 years ago
I made a few tweaks today after implementing my feature branch in production. The trailing /
was throwing things off, but I think I have adjusted adequately for it.
The test failures for puppet 3/no future parser are because I added a type def. Puppet 3 hit EOL on 12/31/2016. I can drop the type def if there's some need to preserve Puppet 3 compatibility, though.
On 03.05.2017 19:14, Rob Nelson wrote:
The test failures for puppet 3/no future parser are because I added a type def. Puppet 3 hit EOL on 12/31/2016. I can drop the type def if there's some need to preserve Puppet 3 compatibility, though. I think there are still some systems out there, which are Puppet 3 (at least I've got one :-)
If it's not a big deal for you... Planning on supporting Puppet 4 only soon.
@saz Sounds good! Updated with no types so it's past_parser compatible.
I also combined two tests in travis to make the matrix a little smaller/faster (for some reason it's not using it right now, but would certainly be used on future PRs). It's what we use in Vox Pupuli projects, works pretty well.
@saz Any update on this?
Bump
I have need to support multiple
#includedir
stanzas, which are dynamic, rather than the single stanza in the file resources used now. This PR converts thefile {'/etc/sudoers/:}
resource from using a static file to an ERB template. Additional stanzas can be included by updating thesudo::extra_include_dirs
key with a list of directories. Here's an example of the effective differences on RHEL7:When upgrading from 3.1.0 to 4.1.0 (just to see some changes)
Given the following hiera global file:
And going from 3.1.0 to 4.1.0+includedir branch: