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
nixos/prometheus-*-exporters: minor fixes #81854
Conversation
The allowed values have changed in bd3319d. 0.15: --log.level="info" Only log messages with the given severity or above. Valid levels: [debug, info, warn, error, fatal] --log.format="logger:stderr" Set the log target and format. Example: "logger:syslog?appname=bob&local=7" or "logger:stdout?json=true" 0.17: --log.level=info Only log messages with the given severity or above. One of: [debug, info, warn, error] --log.format=logfmt Output format of log messages. One of: [logfmt, json]
The assertion was printed when user explicitly defined only the configurationPath option.
The assertion was printed when user explicitly defined only the configFile option.
- Fix misspelled option. mkRenamedOptionModule is not used because the option hasn't really worked before. - Add missing cfg.telemetryPath arg to ExecStart. - Fix mkdir invocation in test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad to see you enjoy the prometheus modules, also thank you very much for the cleanup 👍
Thank you for the swift review & taking care of backporting! |
@WilliButz I realized that 20.03 has the older version of the SNMP exporter so now the module options mismatch those of the package. Should we revert 6673c4f or backport bd3319d? |
Oh, thanks for noticing. The exporter version bump should have already been backported. I'll do that in a moment. |
backported the exporter update in b417adb |
I've been playing with Prometheus and the NixOS modules are very nice! Run into a few problems in the exporters which this PR fixes:
disableFileDelition
for mail exporter is misspelled and has no effectconfiguration
orconfigFile
options do not work as intended, the assertion is triggered when user uses justconfigFile
and doesn't touchconfiguration
unifiPassword
containing space (or filesystem path w/ space, eww)Motivation for this change
NixOS prometheus experience only 98 % smooth.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)cc @WilliButz