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
prometheus-nginx-exporter: switch to official version & general prometheus-exporters cleanup #65088
Conversation
@GrahamcOfBorg test prometheus-exporters.bind prometheus-exporters.blackbox prometheus-exporters.collectd prometheus-exporters.dnsmasq prometheus-exporters.dovecot prometheus-exporters.fritzbox prometheus-exporters.json prometheus-exporters.nginx prometheus-exporters.node |
nixos/modules/services/monitoring/prometheus/exporters/nginx.nix
Outdated
Show resolved
Hide resolved
|
Pass through 'options' to exporter definitions and replace 'with lib;' by explicit function imports.
Replace 'with lib;' by explicit function imports.
Adds the functionality to create option renamings and removals for exporter submodules as in nixos/modules/rename.nix.
Use official exporter from nginxinc.
Update exporter submodule to match the new exporter version.
5e5a11c
to
294bed6
Compare
I included the proposed changes and rebased onto master. |
Motivation for this change
NGINX Inc. provide their own prometheus exporter.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)/cc @fpletz @benley as additional maintainers of the package
prometheus-nginx-exporter