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.

Sign up for GitHub

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

Jump to bottom

feat(api): add prometheus monitoring #1166

Merged
merged 16 commits into from
Oct 3, 2024
Merged

feat(api): add prometheus monitoring #1166

merged 16 commits into from
Oct 3, 2024

Conversation

gphorvath
Copy link

@gphorvath gphorvath commented Sep 30, 2024 •
edited
Loading

Description

Sets up monitoring for API.

BREAKING CHANGES

None

CHANGES

  • Adds a python library for exposing basic fastapi metrics in prometheus syntax on /metrics endpoint
  • Add a policy to block access to the /metrics from outside the cluster
  • Adds a service monitor to api deployment so prometheus will scrape metrics

Related Issue

Relates to #1018

Checklist before merging

@gphorvath gphorvath added the enhancement New feature or request label Sep 30, 2024
@gphorvath gphorvath self-assigned this Sep 30, 2024
@netlify Netlify
Copy link

netlify bot commented Sep 30, 2024 •
edited
Loading

✅ Deploy Preview for leapfrogai-docs canceled.

Name Link
🔨 Latest commit b1a0efc
🔍 Latest deploy log https://app.netlify.com/sites/leapfrogai-docs/deploys/66fdb716aec2e200087f2453

@justinthelaw justinthelaw linked an issue Sep 30, 2024 that may be closed by this pull request
feat: LeapfrogAI and UDS Prometheus Integration #1018
Closed
@gphorvath gphorvath marked this pull request as ready for review October 2, 2024 20:48
@gphorvath gphorvath requested a review from a team as a code owner October 2, 2024 20:48
@gphorvath gphorvath merged commit 81ff6a3 into main Oct 3, 2024
21 checks passed
@gphorvath gphorvath deleted the prometheus-for-api branch October 3, 2024 02:24
@github-actions github-actions bot mentioned this pull request Oct 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CollectiveUnicorn CollectiveUnicorn CollectiveUnicorn approved these changes

@justinthelaw justinthelaw justinthelaw approved these changes

Assignees

@gphorvath gphorvath

Labels
enhancement New feature or request
Projects
None yet
Milestone
Future (M12) - Metrics & Monitoring
Development

Successfully merging this pull request may close these issues.

feat: LeapfrogAI and UDS Prometheus Integration
3 participants
@gphorvath @justinthelaw @CollectiveUnicorn