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

feat: Adding new panels in Disk dashboard #2391

Merged
merged 6 commits into from
Oct 4, 2023
Merged

Conversation

Hardikl
Copy link
Contributor

@Hardikl Hardikl commented Sep 29, 2023

No description provided.

@Hardikl
Copy link
Contributor Author

Hardikl commented Sep 29, 2023

image

@Hardikl Hardikl linked an issue Sep 29, 2023 that may be closed by this pull request
cgrinds
cgrinds previously approved these changes Sep 29, 2023
grafana/dashboards/cmode/disk.json Show resolved Hide resolved
"format": "time_series",
"instant": false,
"interval": "",
"legendFormat": "{{disk}}",
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like only new panels support raid and plex filtering which is not very clear from the UI.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, May be I could add new panes in new row and row name should specify that Top Disks: Raid-level overview.
Any suggestion?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added in separate panels.

@Hardikl
Copy link
Contributor Author

Hardikl commented Oct 3, 2023

image

cgrinds
cgrinds previously approved these changes Oct 3, 2023
"format": "time_series",
"instant": false,
"interval": "",
"legendFormat": "{{cluster}} - {{disk}}",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should raid/flex be displayed in legend?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

@rahulguptajss rahulguptajss merged commit 133545b into main Oct 4, 2023
@rahulguptajss rahulguptajss deleted the hl_disk_change branch October 4, 2023 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create dashboard exactly like Disk View in perf archives
3 participants