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

Added KES grafana dashboard #447

Merged
merged 2 commits into from
Feb 20, 2024
Merged

Added KES grafana dashboard #447

merged 2 commits into from
Feb 20, 2024

Conversation

shtripat
Copy link
Contributor

No description provided.

@shtripat shtripat requested review from aead and kannappanr February 19, 2024 16:51
@aead aead force-pushed the grafana-dashboard branch from df9b98c to cbe69ae Compare February 20, 2024 06:42
@aead
Copy link
Member

aead commented Feb 20, 2024

LGTM @shtripat - I just want to change the location where things are stored.

Can we move:

  • docs/metrics/prometheus/grafana/grafana-kes.png => .github/grafana-dashboard.png
  • docs/metrics/prometheus/grafana/kes-dashboard.json => examples/grafana/dashboard.json

And instead of a separate README, just add a sub-section to the top level README Docs:

#### Monitoring

KES servers provide an API endpoint `/v1/metrics` that observability tools, like Prometheus, can scrape.
Refer to the [monitoring documentation](https://min.io/docs/kes/concepts/monitoring/) for how to setup and capture KES metrics.

For a graphical Grafana dashboard refer to the following [example](examples/grafana/dashboard.json).

![.github/grafana-dashboard.png]

@aead
Copy link
Member

aead commented Feb 20, 2024

cc @djwfyi - We may want to update our KES monitoring docs to include the grafana dashboard

Signed-off-by: Shubhendu Ram Tripathi <shubhendu@minio.io>
Signed-off-by: Shubhendu Ram Tripathi <shubhendu@minio.io>
@aead aead merged commit a9f7817 into minio:master Feb 20, 2024
7 checks passed
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