Skip to content

(infra): Adjust Sampling Rate for Minds Front and Engine minds/infrastructure/minds-terraform#44

Zack Wynne requested to merge infra/enable-tracing-minds-terraform44 into master

Ticket(s) / Related Merge Requests

Issue: minds-terraform#44

Summary of Changes

This MR reduces the sampling rate for the Minds Front and Engine deployments, meaning that approximately one in ten requests will be visible in Jaeger rather than all requests.

I'm also removing the operation declarations for each location block in the NGINX config.

Testing Considerations

The change is live in Sandbox and Production, so you can view the traces in Jaeger:

kubectl port-forward svc/jaeger-operator-jaeger-query -n tracing 16686:16686

Deployment Considerations

We should keep an eye on the resource utilization of Jaeger relative to the requests and limits:

https://dashboards.minds.com/d/wZJ2BxQ7k/requests-limits?orgId=1&refresh=5s&var-namespace=tracing&from=now-5m&to=now

Regression Scope

This change affects the NGINX config for both frontend and engine deployments.

Platform Affected (web, mobile, etc)

Both

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