Remove check for Apache 2.4/2.2, require puppetlabs/apache 9.x

voxpupuli / puppet-zabbix

Puppet module for creating and maintaining zabbix components with puppet.

https://forge.puppet.com/puppet/zabbix

Apache License 2.0

79 stars 228 forks source link

Remove check for Apache 2.4/2.2, require puppetlabs/apache 9.x #859

Closed JoshuaSmeda closed 1 year ago

JoshuaSmeda commented 1 year ago

Pull Request (PR) description

Apache 9.0.0 release drops 2.2 support. Since Zabbix has a dependency on Apache, it makes sense to remove the comparison between >= 2.4 and 2.2, including setting the minimum dependency version for Apache to 9.0.0.

The Zabbix module breaks if you run Apache version higher than 9.0.0, due to not being able to verify the fact.

bastelfreak commented 1 year ago

I think it's fine to to mark this backwards-incompatible. We deprecated CentOS 6 in the past, and that's the only OS that used Apache 2.2.

evgeni commented 9 months ago

While dropping C6 support is fine, bumping puppetlabs/apache is backwards incompatible.