Bug 1613533

Summary: Description of Throughput panel on Brick dashboard states it depends on gluster profiling, but it shown data with disabled profiling anyway
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Martin Bukatovic <mbukatov>
Component: web-admin-tendrl-monitoring-integrationAssignee: gowtham <gshanmug>
Status: CLOSED WONTFIX QA Contact: sds-qe-bugs
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rhgs-3.4CC: mbukatov, nthomas, rhs-bugs, sankarshan, shtripat
Target Milestone: ---Keywords: ZStream
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-05-08 16:11:43 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
screenshot 1: Throughput panel on Brick dashboard with disabled profiling none

Description Martin Bukatovic 2018-08-07 18:52:34 UTC 
Description of problem
======================

Description of Throughput panel on Brick dashboard (Disk Load section) states
that the values are fetched using `gluster volume profile` command, but the
values are shown no matter if the gluster profiling is actually enabled.

Version-Release number of selected component
============================================

tendrl-monitoring-integration-1.6.3-7.el7rhgs.noarch

How reproducible
================

100 %

Steps to Reproduce
==================

1. Instal RHGS WA using tendrl-ansible
2. Import Trusted storage pool with at least one volume, disable profiling
3. Go to Brick dashboard and check description of "Througput" panel
   there (available via little "i" icon on top left corner of the panel).

Actual results
==============

The description states that the profiling information are used:

> The Throughput panel displays brick-level read and write operations
> fetched using “gluster volume profile”.

But the data are shown anyway.

See screenshot 1.

Expected results
================

One of:

 * clarification what this panel actually reports in the description
 * fixing the way this panel fetches data (make it use profiling if that is
   actual intention) - based on information provided under bz 1590693, this
   seems not to be the case
Comment 1 Martin Bukatovic 2018-08-07 18:54:27 UTC 
Created attachment 1474078 [details]
screenshot 1: Throughput panel on Brick dashboard with disabled profiling
Comment 2 Martin Bukatovic 2018-08-07 18:55:13 UTC 
What option proposed in Expected results section is expected?
Comment 3 gowtham 2018-08-08 05:04:37 UTC 
Martin all details under disk_load in brick dashboards are fetched using psutil library, not from volume profiling command. It is not related to volume profile enable. all disk details are collected using mount point.
Comment 4 Martin Bukatovic 2018-08-08 07:03:16 UTC 
gowtham answered the question from comment 2, thank you
Comment 6 Shubhendu Tripathi 2018-11-19 12:54:54 UTC 
Martin, based on https://bugzilla.redhat.com/show_bug.cgi?id=1613533#c4 is this still a valid issue?
Comment 7 Martin Bukatovic 2019-01-23 20:16:17 UTC 
(In reply to Shubhendu Tripathi from comment #6)
> Martin, based on https://bugzilla.redhat.com/show_bug.cgi?id=1613533#c4 is
> this still a valid issue?

I don't see any reason for this issue to be invalid.
Comment 8 gowtham 2019-01-24 09:04:34 UTC 
The description is wrong, we are fetching this information using device names of the brick from the output of psutil.disk_io_counters(perdisk=True)