puppet-nginx
puppet-nginx copied to clipboard
Allow specifying error_log severity for servers
Fixes #1416
Note - same change may be needed in other template files.
Could you add a test for this too? Seems like there's a similar one already here: https://github.com/voxpupuli/puppet-nginx/blob/master/spec/classes/nginx_spec.rb#L495
I think I did it right; also added the same fix for non-TLS server entries. Since I'm not terribly familiar with this another set of eyes would be useful. Also uncertain why there's a failed test here..
Right, now I think I see why it's failing the tests. My original patch used the top level error_log_severity; it did not introduce this as a setting for servers. Hence the tests are testing something that isn't actually there. What would you like me to do about this? I can try to add the parameters to the server defines, hoping I get that right..
Yeah I think adding the severity as a class parameter to https://github.com/voxpupuli/puppet-nginx/blob/master/manifests/resource/server.pp is the way to go, similar to the parameters already there which are taking their value from the base class.