Bug 1614263 - Remove Subvolume panel on Volume dashboard (because current info is insufficient)
Summary: Remove Subvolume panel on Volume dashboard (because current info is insuffici...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Gluster Storage
Classification: Red Hat Storage
Component: web-admin-tendrl-monitoring-integration
Version: rhgs-3.4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: RHGS 3.4.z Batch Update 1
Assignee: gowtham
QA Contact: sds-qe-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-08-09 09:35 UTC by Martin Bukatovic
Modified: 2018-10-31 08:45 UTC (History)
11 users (show)

Fixed In Version: tendrl-monitoring-integration-1.6.3-13.el7rhgs
Doc Type: Bug Fix
Doc Text:
The Subvolume panel in the Volume dashboard of Grafana displayed only the subvolumes count without any explanation of subvolumes or without depicting health status. Since there was no useful metric displayed in the subvolume panel, the panel is now removed from the Volume dashboard
Clone Of:
Environment:
Last Closed: 2018-10-31 08:45:18 UTC
Embargoed:


Attachments (Terms of Use)
screenshot 1: subvolume panel with it's description (21.54 KB, image/png)
2018-08-09 09:39 UTC, Martin Bukatovic
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github Tendrl monitoring-integration issues 570 0 None None None 2018-09-11 13:36:38 UTC
Red Hat Bugzilla 1627828 0 unspecified CLOSED [RFE] Add Subvolume panel on Volume dashboard with health state 2023-09-14 04:34:40 UTC
Red Hat Product Errata RHBA-2018:3427 0 None None None 2018-10-31 08:45:49 UTC

Internal Links: 1627828

Description Martin Bukatovic 2018-08-09 09:35:18 UTC
Description of problem
======================

Description of Subvolume panel on Volume dashboard doesn't explain what
gluster subvolume is. Moreover it talks about some status info, while
only total number is actually provided on the panel.

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 Subvolume panel's description (available
   via little "i" icon on top left corner of the panel).

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

The panel reports just number of subvolumes.

The description states:

> The Subvolumes panel displays subvolume status information for a given volume.

See screenshot 1.

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

The description explains what a gluster subvolume is, so that customer would
be able to understand information reported by this panel.

Moreover, the description mentions status information, but I see only a total
number reported on the panel. If this is the only information provided here,
the description should clearly articulate that.

Comment 1 Martin Bukatovic 2018-08-09 09:39:17 UTC
Created attachment 1474618 [details]
screenshot 1: subvolume panel with it's description

Comment 3 gowtham 2018-09-06 09:00:41 UTC
Proposed Description: 

The Subvolumes panel displays the number of subvolumes in a given volume. A subvolume is a set of bricks which form a replica set for the volume

Comment 4 gowtham 2018-09-06 09:11:15 UTC
Above description may be wrong,

we can go with,

The Subvolumes panel displays the number of subvolumes in a given volume. A brick after being processed by at least one translator or in other words set of one or more xlator stacked together is called a sub-volume.

Comment 5 Ju Lim 2018-09-07 15:15:35 UTC
This is more than a tooltip BZ.  In looking at the Subvolume panel on the Volume Dashboard, it’s current showing # subvolumes and not giving status information.  MartinB asked the relevance of showing # subvolumes.  

The original intent was to be able to show the count along with the health of the subvolumes (e.g. ok, degraded, not ok or the values that we get from Gluster or derive based on calculating this information) in order to identify if there are availability concerns.  

If we’re only showing # subvolumes, it’s not really useful to show it.  However, if we can show the health information of the subvolumes, which was the original intent, then I recommend we fix this accordingly.

RE Comment3 and Comment4 above, it is only applicable if the volume is not an EC volume, and we'd probably also need to state when it is not applicable.

Comment 6 Anand Paladugu 2018-09-09 17:52:34 UTC
I agree with Ju's comments.

Comment 7 gowtham 2018-09-10 06:14:38 UTC
need to discuss this with Nisanth

Comment 9 Ju Lim 2018-09-10 16:57:04 UTC
This is more than a tooltip BZ.  In looking at the Subvolume panel on the Volume Dashboard, it’s current showing # subvolumes and not giving status information.  MartinB asked the relevance of showing # subvolumes.  The original intent was to be able to show the count along with the health of the subvolumes (e.g. ok, degraded, not ok) in order to identify if there are availability concerns.  If we’re only showing # subvolumes, then I agree it’s not useful to show it, and therefore should remove it.

Per discussions, we would prefer to get the health information of the subvolumes from the underlying Gluster, which does not provide this information (vs. calculating it in WA/tendrl).  Therefore, let's remove this panel for now.

Comment 10 Martin Bukatovic 2018-09-11 09:02:36 UTC
I agree with Ju that we should resolve this problem by removing this panel and
create another BZ which swill track adding this panel as original intended, with
info which makes sense.

Comment 14 gowtham 2018-09-11 13:37:23 UTC
PR is under review: https://github.com/Tendrl/monitoring-integration/pull/571

Comment 15 Martin Bukatovic 2018-09-11 15:56:16 UTC
(In reply to Nishanth Thomas from comment #8)
> we need to create a separate RFE BZ and priitize that in a later release

The RFE bugzilla was created as BZ 1627828

Comment 17 Elena Bondarenko 2018-09-25 13:00:57 UTC
There's no Subvolume panel on Volume dashboard now.

Comment 20 gowtham 2018-10-23 11:23:59 UTC
Looks good to me

Comment 23 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


Note You need to log in before you can comment on or make changes to this bug.