Alexander Michelsen

This has already been merged, but .. you can easily support both string and array in the `sql` parameter by doing `flatten([$sql])`. Would have saved some needless breakage. ```Error while...

This default is also an issue on RedHat/CentOS 7, which also uses the predictable interface naming scheme. From `openvpn::server`: ``` String $local = $facts['ipaddress_eth0'], ``` Which results in ``` Error...

Threw in some additions here that fix the remainder of the tests for me: https://github.com/ekohl/puppet-zabbix/pull/1

Ran into this again today. Turns out `options => "http-proxy=\"${proxy}\"",` has no effect for the `source` parameter, only keyservers with `server`. Only workaround I could find was to run Puppet...

This one hit us too. I have a PR in the works but have to wait for this one to be merged first (fixes CI tests): https://github.com/voxpupuli/puppet-zabbix/pull/834

There's some weird stuff going on with `$php::globals::globals_php_version` wanting to set invalid default versions (like `5.x`). It probably wasn't intended to be used outside of the class, but there is...

I wouldn't mind cleaning stuff up. I'd argue I'm pretty good at that. Getting started with acceptance testing using `CONTRIBUTING.md`, and I've run into some problems: ``` $ bundle install...

@bastelfreak Thanks! That helped me get things sorted out. Much appreciated. :) This should be ready for merge now, if everything looks okay.