CERIT-SC / puppet-serial_console

Puppet module for system serial console setup
MIT License
1 stars 8 forks source link

Support Debian 8 (systemd-based) and some misc fixes. #2

Closed sbakker closed 9 years ago

sbakker commented 9 years ago

Hi,

I have a few fixes here that allow for Debian 8 support (not that difficult, just don't try to update /etc/inittab).

As I was going along, I noticed there were variables with a leading underscore. Puppet 4 no longer allows those, so I figured I'd save some work later on and changed them as well.

While I was testing the module using Hiera data, I also noticed that the speed, runlevels, bootloader_timeout and logout_timeout parameters could not be specified as numbers (at least under Puppet 3.7), so I changed the validation to first coerce them into strings ("${speed}") and then run the validation on them.

Finally, I added the term_type parameter, because I prefer vt102 over vt100 :-)

Hope you find this useful.