Romain Tartière
Romain Tartière
After rotation: ```sh-session root@xxxxxxx /var/log/puppetlabs/puppetserver # ls -lirt | tail 102111681 -rw-r-----. 1 puppet puppet 294410 5 avril 00:00 puppetserver-access-2021-04-04.0.log.gz 102113472 -rw-r-----. 1 puppet puppet 369200 5 avril 00:00 puppetserver-2021-04-04.0.log.gz...
Do you confirm this is some WIP for future integration of Arch as a first class citizen in our modulesync config? If so, may I suggest converting this to a...
Maybe it would be better to use a `Variant[String,Struct[...]]` to accommodate possible future parameters? ```puppet class { 'nginx': # ... log_format => { json_combined => { escape => 'json', format...
> This seems a little complex, suggest adding an example to the README. The data type is also lacking documentation. Maybe we can add examples at this level, I'll try...
Superseded by #1513. I completely forgot I already opened a PR for it.
Hi! Can you add some test in your PR to ensure this new feature is not broken by future evolution/refactoring of the module?
> If anyone does try the new functionality, feedback is most definitely welcome. @fatmcgav I just did a quick deployment on Debian 10 (nothing fancy, just a quick "first run"...
@cocker-cc just released [yaml-sort](https://github.com/smortex/yaml-sort) if you want to give it a try :stuck_out_tongue_winking_eye:
> This looks very well implemented. Is there a Place to discuss it? @cocker-cc I opened issues on the project page ;-)
It makes sense! @olevole this has been lagging for a tremendous amount of time, and the module was recently adopted by voxpupuli and gained a refresh! Would you be interested...