Skip to content
This repository has been archived by the owner on Apr 28, 2020. It is now read-only.

Utilization: Disk IO RW chart #332

Merged
merged 1 commit into from
Apr 16, 2019

Conversation

mareklibra
Copy link
Contributor

Implemented as "best-effort", I do not have access to a Ceph environment.

@mareklibra
Copy link
Contributor Author

Depends on: #322

@mareklibra mareklibra force-pushed the utilization.diskIOPS branch from cd50d0b to 4d1ef0a Compare April 13, 2019 10:03
@mareklibra mareklibra changed the title WIP Utilization: Disk IO RW chart Utilization: Disk IO RW chart Apr 13, 2019
@coveralls
Copy link

coveralls commented Apr 13, 2019

Pull Request Test Coverage Report for Build 1506

  • 10 of 10 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 87.35%

Totals Coverage Status
Change from base Build 1273: 0.03%
Covered Lines: 3426
Relevant Lines: 3759

💛 - Coveralls

@atiratree
Copy link
Contributor

missing tests/snapshots

@mareklibra mareklibra force-pushed the utilization.diskIOPS branch 2 times, most recently from 724a867 to d016e5b Compare April 16, 2019 10:39
@kubevirt-bot kubevirt-bot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L and removed size/M labels Apr 16, 2019
@@ -13,4 +13,5 @@ export const STORAGE_PROMETHEUS_QUERIES = {
UTILIZATION_STORAGE_USED_QUERY: 'ceph_cluster_total_used_bytes[60m:5m]',
UTILIZATION_STORAGE_USED_DEFAULT_QUERY:
'(sum(node_filesystem_avail_bytes) - sum(node_filesystem_free_bytes))[60m:5m]',
UTILIZATION_STORAGE_IORW_QUERY: '(sum(rate(ceph_pool_wr_bytes[1m]) + rate(ceph_pool_rd_bytes[1m])))[60m:1m]',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cloudbehl , @gnehapk , @nthomas-redhat, can you please confirm this query properly shows the expected Disk IO RW utilization, please?

@mareklibra mareklibra force-pushed the utilization.diskIOPS branch from d016e5b to 97cce08 Compare April 16, 2019 11:11
@kubevirt-bot kubevirt-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 16, 2019
@rawagner rawagner merged commit 8d14829 into kubevirt:master Apr 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants