-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Add /devices/{id}/statistics path to Swagger config This is in preparation to fetch ping statistics from UNMS. The endpoint returns A LOT more data, but for now this contains only the minimal set. * Add ping metrics This enables fetching device statistics from UNMS, extracting the ping RTT values from it, and export them back to prometheus. Since fetching statistics requires talking to another, device-specific endpoint, the context timeout is increased from 5 to 30s. A future change should pass the request context from the promhttp endpoint down to the fetchDeviceData() method, to make the timout depend on Prometheus' scrape request life cycle. * Ping metrics: disable by default, add config knob * Ping metrics: update README * Make examples in README consistent * Ping metrics: cleanup dead code * Ping metrics: add tests for env vars/CLI flags * Allow usage of dash in metrics option * Update docs and extend test case Co-authored-by: Marcus Weiner <marcus.weiner@gmail.com>
- Loading branch information
Showing
18 changed files
with
1,275 additions
and
10 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Oops, something went wrong.