Bug 1614270

Summary: Description of IOPS panel in Volume refers to all volumes even though it's concerned with single volume only
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: Martin Bukatovic <mbukatov>
Component: web-admin-tendrl-monitoring-integrationAssignee: gowtham <gshanmug>
Status: CLOSED ERRATA QA Contact: Elena Bondarenko <ebondare>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rhgs-3.4CC: apaladug, ebondare, mbukatov, nthomas, rhinduja, rhs-bugs, sankarshan
Target Milestone: ---Keywords: ZStream
Target Release: RHGS 3.4.z Batch Update 1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tendrl-monitoring-integration-1.6.3-13.el7rhgs Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-10-31 08:45:18 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: description of IOPS panel on volume dashboard none

Description Martin Bukatovic 2018-08-09 09:53:16 UTC
Description of problem
======================

The description refers to all volumes, even though this panel is concerned
with a single volume only.

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

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

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

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

Note: the IOPS panel is in Performance section of the Volume dashboard.

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

The description states:

> The IOPS panel displays aggregated brick level read and write operations
> collected using “gluster volume profile info” over a period of time. The
> IOPS value shown only reflects metrics from volumes that have volume
> profiling enabled. No data will be shown if volume profiling is not enabled
> on at least 1 volume.

Which is a little misleading, as this panel is part of Volume dashboard and
so it reports data for particular volume only.

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

Sentence:

> No data will be shown if volume profiling is not enabled
> on at least 1 volume.

Is replace with:

> No data will be shown if volume profiling is not enabled
> on this particular volume.

Or something more polished conveying the same message.

Comment 1 Martin Bukatovic 2018-08-09 09:57:53 UTC
Update of Expected results
==========================

This sentence from the description:

> The IOPS value shown only reflects metrics from volumes that have volume
> profiling enabled.

could be also tweaked to make it more clear that data will be shown only if
the selected volume has profiling enabled.

Comment 2 Martin Bukatovic 2018-08-09 09:58:45 UTC
Created attachment 1474622 [details]
screenshot 1: description of IOPS panel on volume dashboard

Comment 4 gowtham 2018-09-05 15:40:50 UTC
Proposed Description: 


 The IOPS panel displays aggregated brick level read and write operations collected using “gluster volume profile info” over a period of time. The IOPS value shown only reflects metrics from volumes that have volume profiling enabled. No data will be shown if volume profiling is not enabled for the selected volume.

Comment 5 gowtham 2018-09-10 06:15:55 UTC
Martin is above description is fine

Comment 6 Nishanth Thomas 2018-09-10 08:46:42 UTC
Providing ack based on  https://bugzilla.redhat.com/show_bug.cgi?id=1614270#c4

Comment 8 gowtham 2018-09-11 13:50:34 UTC
PR is under review: https://github.com/Tendrl/monitoring-integration/pull/560

Comment 9 gowtham 2018-09-12 07:51:54 UTC
Modified Proposed Description: 


 The IOPS panel displays aggregated brick level read and write operations collected using “gluster volume profile info” over a period of time. The IOPS value shown only reflects metrics from volumes that have volume profiling enabled. No data will be shown if volume profiling is disabled for the selected volume.

Comment 11 Martin Bukatovic 2018-09-25 13:46:30 UTC
(In reply to gowtham from comment #9)
> Modified Proposed Description: 
>
>  The IOPS panel displays aggregated brick level read and write operations
> collected using “gluster volume profile info” over a period of time. The
> IOPS value shown only reflects metrics from volumes that have volume
> profiling enabled. No data will be shown if volume profiling is disabled for
> the selected volume.

Looks good. I see you haven't tweaked the other part of the text as noted in
comment 1, but that was just a suggestion.

Comment 12 Elena Bondarenko 2018-09-26 06:55:27 UTC
The IOPS panel description has been changed according to Comment 9.

Comment 14 errata-xmlrpc 2018-10-31 08:45:18 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:3427