jenkinsci/puppet-jenkins — pull request #22 by tmclaugh (2013-08-13 20:44:01+00:00)
Title: Firewall resource order

Fixes:
- error when configure_firewall is false.
- missing relationship between Class['jenkins::config'] and Class['jenkins::plugins']

FWIW, I don't agree with defaulting configure_firewall to true. The module makes an assumption that I want it to manage my firewall which I think it has no business messing with. You really can't assume how the module consumer wants their firewall to look. There's of course no clear consensus on this idea and I've found about a 50/50 split between those modules that manage the firewall by default and those that don't.