Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Add v1 dashboard #141

Merged
merged 5 commits into from
Jun 8, 2020
Merged

Add v1 dashboard #141

merged 5 commits into from
Jun 8, 2020

Conversation

zmalik
Copy link
Contributor

@zmalik zmalik commented Jun 3, 2020

This PR:

  • adds a v1 dashboard and removes SLA and adds HEALTH section
  • all dashboard uses namespace in their queries so we don't mix different namespace metrics. Move cluster above datacenter variable
  • adds a simple workload.yaml file to test with kubectl kudo install cassandra in default namespace

Dashboard:

image

Copy link
Contributor

@porridge porridge left a comment

Choose a reason for hiding this comment

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

Please consider varying the colors between the heap/CPU/disk/... graphs. This provides a sub-conscious visual cue about which graph is which - after some usage the brain learns which graph is which and does not need to spend time reading the descriptions to remember.

I have a graph like this on my desktop panel and can vouch for the value of this convention:
Zrzut ekranu z 2020-06-04 06-48-50

@zmalik
Copy link
Contributor Author

zmalik commented Jun 4, 2020

thanks @porridge I think this is a great idea and I will update the dashboard

@zmalik zmalik force-pushed the zain/v1-dashboard branch from 9bf0dec to c70e1aa Compare June 5, 2020 10:38
@ANeumann82 ANeumann82 merged commit 2c8b74e into master Jun 8, 2020
@ANeumann82 ANeumann82 deleted the zain/v1-dashboard branch June 8, 2020 17:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants