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

fix grafana dashboard for keycloak 23.0.7 / keycloak-spi-metrics 5.0.0 #196

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

Conversation

ptrovatelli
Copy link

@ptrovatelli ptrovatelli commented Feb 29, 2024

What

  • Put grafana dashboard under version control
  • Fix dashboard for keycloak 23.0.7 / keycloak-metrics-spi 5.0.0

Changes:

  • switch from graph (angular js deprecated) to timeseries in several widgets
  • in widget and in variables, fix field names : usually needed to add total :
    • keycloak_logins -> keycloak_logins_total
    • keycloak_response_errors -> keycloak_response_errors_total
    • keycloak_registrations -> keycloak_registrations_total
    • keycloak_failed_login_attempts -> keycloak_failed_login_attempts_total
    • keycloak_registrations_errors -> keycloak_registrations_errors_total
    • ...
  • in widgets, update description to state it's for all instances when there is no instance filter

Checklist

I had to manipulate a little bit the dashboard before publishing it as we have specificities that I needed to strip out. Consequently the version I pushed here is not 100% error proof.

Additional Notes

On openshift, replace kubernetes_pod_name by pod_names

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.

None yet

1 participant