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

Om135 #81

Merged
merged 8 commits into from
Nov 7, 2023
Merged

Om135 #81

merged 8 commits into from
Nov 7, 2023

Conversation

mphanias
Copy link
Contributor

added panel to show aerospike-version
modified dashboard to show 7.0 metrics along with 6.0

modified dashboard according to 7.0 metrics
modified dashboard to support 7.0. metrics along with 6.0
added panel to show aerospike-versions
@mphanias mphanias requested review from sunilvirus and hev October 23, 2023 08:53
@hev
Copy link
Collaborator

hev commented Oct 24, 2023

@mphanias #80 also changes all-flash. Can we close one of these to consolidate?

modified Migration panel to include rx/tx active
adjust panel colors to same threshold values
1. updated panel titles and labels
2. updated time-series legends to last, min, max and mean
reverted all-flash dashboard changes from this pr as all-flash is handled as part of OM-134
1. fixed panel titles and order
2. fixed data-types
3. removed unnecessary query
Copy link
Collaborator

@hev hev left a comment

Choose a reason for hiding this comment

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

LGTM

added description
included topk and bottok in data used vs total panel as per OM-114
added bottomk in memory and data free,used/available panels also
updated labels to represent 5x/6x and 7x server versions
added descriptions in few panels
@sunilvirus sunilvirus merged commit cd492ad into dev Nov 7, 2023
1 check passed
@sunilvirus sunilvirus deleted the OM135 branch November 7, 2023 06:31
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