Closed henrythasler closed 1 month ago
@hansmi, I have refactored the PR quite a bit according to your your suggestions. Feel free to review again.
@henrythasler, I'll be taking a look at the code changes as a separate review. Generally I'd suggest to break out the "fix homeassistant service discovery" changes to a separate pull request.
@hansmi, you are correct. I will create a new PR for the homeassistant change.
Please also extend the documentation at https://github.com/jomjol/AI-on-the-edge-device-docs/blob/main/docs/REST-API.md
Please also extend the documentation at https://github.com/jomjol/AI-on-the-edge-device-docs/blob/main/docs/REST-API.md
Done: https://github.com/jomjol/AI-on-the-edge-device-docs/pull/55
Adds a new http-endpoint
/metrics
that provides metrics according to Prometheus (similar to OpenMetrics) text format. Fixes #1345. The metrics mostly follow the design of @hansmi's Prometheus Exporter.The main metric is probably the current value (ai_on_the_edge_device_flow_value) of the main sequence. Multiple sequences are supported via tags. This metric only provides valid readout (
no error
). Besides that, a bunch of device properties are exposed.I'm also creating this PR to get some early feedback and advice from the community on what could be improved.
Simple Prometheus Scrape Config
Sample response
Resource Impact