Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Turnpike slo sli #80

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Conversation

mjholder
Copy link

@mjholder mjholder commented Aug 2, 2022

SLO/SLI for turnpike:
This PR adds two sets of metrics for prometheus to scrape:

  1. Auth plugin metrics.
    • Metrics exposed on localhost:5000/metrics
    • Metrics consist of response time and response code labeled per plugin
  2. Nginx metrics.
    • Metrics exposed on localhost:24321/metrics
    • Metrics read from access logs of nginx which are written to a shared volume for fluentd to read
    • Metrics are labeled by method (GET, POST, etc) and status (200, 400, etc):
      • Upstream response time
      • request total duration
      • Bytes sent

To test:

  • get turnpike running (docs
  • upon hitting https://turnpike.example.com/api/turnpike/identity/ or any other turnpike endpoint you should see logs from web_1 and fluentd_1 printing metrics
  • go to the endpoints localhost:5000/metrics and localhost:24321/metrics to see the metrics being exposed for prometheus.
  • if you have a local prometheus running you can configure it to scrape those endpoints

@mjholder mjholder changed the title Draft: Turnpike slo sli Turnpike slo sli Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants