The apache::monitoring module installs Diamond's httpd collector, we should make sure the machines with such module also end up installing prometheus::apache_exporter.
|Resolved||fgiunchedi||T177195 Reduce technical debt in metrics monitoring|
|Resolved||fgiunchedi||T177196 Port non-deprecated Diamond collectors to Prometheus|
|Open||None||T187434 Include apache_exporter in puppet module httpd (was: apache)|
But we also want to replace the apache module with the httpd module (which doesn't have monitoring.pp anymore because we didn't want the diamond collector). So it should be added there as well.
Indeed, so the current way we are using apache_exporter in puppet is via profile::prometheus::apache_exporter which includes prometheus::apache_exporter and ferm rules. We could move to a profile::httpd which sets up the httpd class and the apache_exporter profile above.