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

Cluster: IOPS in B/s unit #202

Closed
matejzero opened this issue Jun 21, 2021 · 8 comments · Fixed by #205
Closed

Cluster: IOPS in B/s unit #202

matejzero opened this issue Jun 21, 2021 · 8 comments · Fixed by #205
Labels
bug Something isn't working status/done

Comments

@matejzero
Copy link

Describe the bug

In cluster dashboard, IOPS are shown as Bytes/s unit.

Environment

Harvest version: harvest version 21.05.2-1 (commit ce091de) (build date 2021-06-14T20:31:09+0530) linux/amd64
Command line arguments used: /opt/harvest/bin/harvest restart --config /opt/harvest/harvest.yml
OS: CentOS 7
Install method: yum
ONTAP Version: 9.7
Other:

To Reproduce

Open Cluster dashboard and look at IOPs graph.

Expected behavior

Unit should be iops/s or none.

Actual behavior

Screenshot 2021-06-21 at 15 08 54

Possible solution, workaround, fix

Change unit to IO ops/sec

@cgrinds cgrinds added bug Something isn't working and removed status/needs-triage labels Jun 21, 2021
@cgrinds
Copy link
Collaborator

cgrinds commented Jun 21, 2021

@Hardikl can you take a look? - this is similar to #186

@Hardikl
Copy link
Contributor

Hardikl commented Jun 21, 2021

yes, it is same as #186, just it was targeted for node dashboard.
Found few other graphs in other dashboard also shows wrong unit. I will cover throughput, iops and latency related graphs in all dashboards and handle them here.

@ruanruijuan
Copy link

Thank you Hardik for being proactive on examining these counters on all dashboards!

@Hardikl Hardikl linked a pull request Jun 21, 2021 that will close this issue
@sridevimm
Copy link
Contributor

Tested this scenario in Harvest point release 21.05.3.1 and able to reproduce this issue. Cluster IOPs chart still shows unit as kB/s instead of iops/s. Reopening the issue to get this fixed.

Screen Shot 2021-06-23 at 11 55 30 AM

@sridevimm
Copy link
Contributor

Node title shows K io/s. But, the charts are showing millisecs and microsecs. Please take a look at the units for those charts as well.

Screen Shot 2021-06-23 at 12 03 41 PM

@rahulguptajss
Copy link
Contributor

@Hardikl do check once if the test setup had latest grafana dashboards

@Hardikl
Copy link
Contributor

Hardikl commented Jun 24, 2021

image

image

I have installed the latest binary in RPM[10.140.132.80 root/netapp], alongwith the grafana(8.0.3) and prometheous(2.28.0).
Then imported the dashboards via this:
$ bin/grafana import --addr my.grafana.server:3000 --directory grafana/prometheus

Cluster and node dashboard graphs are showing as expected. Also I have checked that the changes are available in release branch as well, so moving this to testme.

@Hardikl
Copy link
Contributor

Hardikl commented Jun 24, 2021

@rahulguptajss out of 2 rhel harvest qa deployment, i was not able to access the grafana UI, I am not sure that test setup has latest grafana dashboards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status/done
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants