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/py-air-control: init #106067
nixos/prometheus-exporters/py-air-control: init #106067
Conversation
|
||
in | ||
{ | ||
port = 9896; |
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.
This port has been added to the Prometheus' port allocations table.
66d6aa0
to
8a82e75
Compare
Added tests. They will fail until the Edit: the package was updated and the tests now pass. @WilliButz: Is there a way to trigger these new tests in CI? Here's the way I ran them:
|
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.
LGTM, just one minor remark 👍
nixos/modules/services/monitoring/prometheus/exporters/py-air-control.nix
Outdated
Show resolved
Hide resolved
369dcf2
to
2bbf895
Compare
2bbf895
to
4948743
Compare
@GrahamcOfBorg test prometheus-exporters.py-air-control |
Motivation for this change
This change adds the
py-air-control
Prometheus Exporter Module for NixOS.This module sets up the
prometheus-py-air-control-exporter
service which gathers air quality information from an air quality/air purification device connected to the network.This is a follow-up to the PR #105966.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
) (tested the execution of this newprometheus-py-air-control-exporter
service on a Raspberry Pi with an actual air purifier device)nix path-info -S
before and after)cc @SuperSandro2000 @lukegb