Remove firewalld package

[tommyj]

If present, firewalld must be removed on RHEL 7 and Fedora systems, because it and the puppetlabs firewall will clobber each other. Unfortunately firewalld is installed by default even on minimal installations. So we must take care to ensure that it is removed.

I release this change to the public domain.

[cfeskens]

This would be very handy for RHEL 7 support. +1

[treydock]

+1 but make it apply to CentOS 7 too. Could the two conditions for RedHat and Fedora be combined into one that does something like

  if $::osfamily == 'RedHat' and versioncmp($::operatingsystemrelease, '7.0') >= 0 {
    package { 'iptables-services':
      ensure => present,
    }
    package { 'firewalld':
      ensure => absent,
    }
  }

[jeremyvisser]

@treydock Given there has been no response from the O.P., you might want to put in your own pull request?

[cfeskens]

Thank you!