Alan-Jowett / CoPilot-For-Consensus / 20150697587

82%

push

github

Switch ingestion metrics to Pushgateway (#96)

* Switch ingestion metrics to pushgateway

* Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Add tests for PrometheusPushGatewayMetricsCollector and fix skip condition

* PR feedback

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Update ingestion/test_integration.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix metrics tests and add can_push attribute

- Add can_push = True class attribute to PrometheusPushGatewayMetricsCollector for proper feature detection in callers
- Fix skipif condition and enhance docstring for test_requires_prometheus_client (skipped when prometheus_client is installed, runs only when missing)
- Use management_ok result in test_integration.py prerequisite checks to avoid unused variable warning

---------

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

20 of 37 new or added lines in 3 files covered. (54.05%)

104 of 183 relevant lines covered (56.83%)

0.57 hits per line