Skip to content

(feat): Engine Metrics Endpoint minds/engine#3183

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

Ticket(s) / Related Merge Requests

Issue: engine#2328 (closed)

Summary of Changes

This MR adds the metrics endpoint for the engine (introduced in this MR to the PodMonitor resource for this chart.

I'm also allowing an additional ingress port on the engine's network policy to allow Prometheus to scrape the endpoint.

Testing Considerations

For Sandbox, we can port-forward Prometheus and attempt to query for the metrics added by the endpoint.

kubectl port-forward svc/prometheus-operated -n monitoring 9090:9090
curl http://localhost:9090/api/v1/query?query=engine_cassandra_available_connections_total

Deployment Considerations

We should deploy the engine changes before updating the Helm chart.

Regression Scope

This should affect the engine deployment in EKS.

Platform Affected (web, mobile, etc)

API

Developer Testing Completed

Deployed changes to Sandbox and tested querying metrics.

Screenshots / Screen Recording

Screen_Shot_2022-06-10_at_12.47.04_PM

Screen_Shot_2022-06-10_at_12.47.57_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

Merge request reports