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

fleet manager prometheus metrics #533

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

venkataanil
Copy link
Collaborator

@venkataanil venkataanil commented Feb 9, 2023

SRE is using these prometheus metrics in their grafana dashboards [1]. After fleet manager scale testing, we manually run kube-burner with this metric profile to index into our internal ES. We have internal grafana dashboard [2] which uses these metrics from our ES.

[1] https://grafana.app-sre.devshift.net/d/osd_fleet_manager1/osd-fleet-manager-metrics1 [2] https://grafana.rdu2.scalelab.redhat.com:3000/d/osd_fleet_manager1/osd-fleet-manager-metrics1

SRE is using these prometheus metrics in their grafana dashboards
[1]. After fleet manager scale testing, we manually run kube-burner
with this metric profile to index into our internal ES.
We have internal grafana dashboard [2] which uses these metrics
from our ES.

[1] https://grafana.app-sre.devshift.net/d/osd_fleet_manager1/osd-fleet-manager-metrics1
[2] https://grafana.rdu2.scalelab.redhat.com:3000/d/osd_fleet_manager1/osd-fleet-manager-metrics1
@venkataanil
Copy link
Collaborator Author

Initially submitted this file in perfscale-managed-services repo with PR cloud-bulldozer/perfscale-managed-services#110 . However I got suggestions to submit in e2e-benchmarking rather than perfscale-managed-services.

Copy link
Member

@krishvoor krishvoor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@@ -0,0 +1,238 @@
- query: 'sum(fleet_manager_management_cluster_hosted_cluster_count{ job="fleet-manager-metrics", namespace="osd-fleet-manager-stage", region="us-east-2"}) by (management_cluster)'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will it always be us-east-2 ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest just having this be a template where we use envsub to swap out the region

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we got us-east-2 for our testing. I will update it to use envsub

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.

3 participants