-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move Apache exporter https://github.com/Lusitaniae/apache_exporter to community #47
Comments
Do you have specific concerns with the apache exporter being independent? I wonder if we should move the exporter to the community without knowing what the future would bring - the exporter could be replaced by native solutions like https://github.com/ConSol/apache_mod_prometheus_status in the future. I'd also note that I have commit access to the apache_exporter, even if I don't actively maintain it at the moment. |
No there is no particular concern, but rather the impression that exporters in the community are aligned in their use of e.g. the exporter toolkit and features like support for TLS and auth. Nothing that cannot be achieved independently no, but I simply appreciate that the community exporters are treated in this way. Thanks for the hint about https://github.com/ConSol/apache_mod_prometheus_status - having native metrics support without the need to run an exporter is nice - Just like HAProxy does it. But this still just being a module will likely make this an option, not a full replacement for an exporter to mod_status. Some installs simply might not be able or want to add modules to their Apaches. |
To echo what @frittentheke said I think there are benefits to having the exporter under the prometheus-community namespace and it allows the author to easily walk away from the project if they no longer wish to maintain it in the future without causing a potential disruption for users. |
@Lusitaniae did say he was open to move the exporter over and also believes there could be benefits, see Lusitaniae/apache_exporter#145 (comment). I understand @roidelapluie that prometheus-community cannot simply adopt ever exporter out there. But frankly the Apache webserver is really extremely common. While I also believe in adding native Prometheus metrics support, as services like HAProxy, RabbitMQ and others did implment for their software and what would be possible with https://github.com/ConSol/apache_mod_prometheus_status this is still an "external" module (read: not something natively provided by Apache Httpd). So even some time down the road not everybody is comfortable adding such a module and rather use the well aged mod_status and scrape that scoreboard via an exporter. |
@roidelapluie @SuperQ any update on considering this request? |
Hey @SuperQ, I've managed the last four releases of https://github.com/Lusitaniae/apache_exporter/releases and my company no longer uses the exporter so it would be best if the community adopts it so that it can stay maintained. |
I'd like to propose moving the popular apache exporter to this community. I bluntly brought up the question with the creator @Lusitaniae in issue Lusitaniae/apache_exporter#145 and seems to like the idea.
With this being the de-facto standard exporter for Apache (see https://prometheus.io/docs/instrumenting/exporters/#http) already and Apache still being all over the place I believe it would be great addition.
The text was updated successfully, but these errors were encountered: