Skip to content

(feat): Moving Prometheus Counters to Gagues #2328

Zack Wynne requested to merge feat/connectivity-metrics-2328 into master

Ticket(s) / Related Merge Requests

Issue: #2328 (closed)

Summary of Changes

This MR moves the metrics being collected from the engine to gauges rather than counters. This allows the values to increase and decrease rather than being continuously incremented. This is needed to alert on the connections dropping to 0 for example.

Testing Considerations

We can test by port-forwarding the Prometheus installation in Sandbox and querying the metrics endpoint for the engine.

Deployment Considerations

This shouldn't require any infrastructure changes, the new metrics will automatically be scraped by Prometheus after deployment.

Regression Scope

This should only affect the metrics endpoint for the engine.

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

Deployed changes to sandbox and tested pulling metrics for the deployment.

Screenshots / Screen Recording

Example with Cassandra

Screen_Shot_2022-06-21_at_9.04.44_PM

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Zack Wynne

Merge request reports