Project

General

Profile

Bug #28870

Improve label of iSCSI target graph and improve Disk Reporting page in new UI

Added by Stilez y over 2 years ago. Updated almost 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Damian Szidiropulosz
Category:
GUI (new)
Target version:
Seen in:
Severity:
Med High
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

See screenshot.

User wishes to watch diskIO (as one might wish to do).

  • First problem is, the UI limits the display to 20 graphs, but even a moderate size pool generates about 5 graphs per disk, so a pool of 10 - 15 HDD/SSDs will generate 70+ graphs. The UI then prevents a user from watching them simultaneously, even as small sized graphs, and there is no kind of higher level summary graph to use instead.
  • Second problem is that even on a decent platform (I'm running a hexacore client and quad core Xeon v4 NAS with tons of RAM and not a lot else going on), the prettiness of the graphs requires enough CPOU that apparently something cannot cope. It shouldn't have that issue but apparently it does. So even the few graphs/partial reporting set I can see, aren't actually rendering happily or at all.

The reporting side badly needs a different approach that can cope with moderate 10 - 15 device pools, and it should be able to handle more. It should probably provide a tabular version or a set of graphs that are less intensive to render (who cares if "pretty"!) and perhaps highger level such as vdev or "customised" selections. ("gstat" and "zfs iostat" can do it, this should be able to as well.)


Related issues

Related to FreeNAS - Bug #52151: Reporting - shows max 5 graphsClosed
Related to FreeNAS - Bug #53748: SCSI target port graphs in new UIClosed
Related to FreeNAS - Bug #53566: Fix iSCSI target Reporting label in new UIDone
Has duplicate FreeNAS - Bug #28868: Fix disk reporting graphClosed

History

#1 Updated by Stilez y over 2 years ago

  • File deleted (temp-test-server-iograph)

#2 Updated by Stilez y over 2 years ago

14572

#3 Updated by Dru Lavigne over 2 years ago

  • Assignee changed from Release Council to Erin Clark
  • Target version set to 11.2-RC2

#4 Updated by Erin Clark over 2 years ago

  • Assignee changed from Erin Clark to Damian Szidiropulosz

#5 Updated by Damian Szidiropulosz over 2 years ago

  • Status changed from Not Started to Screened
  • Severity set to Med High

This will be addressed as part of a broader optimisation effort of the c3.js library and how the reporting page is put together in general. Currently the navigation is accomplished using tabs. Unfortunately the tabs don't destroy the content when they aren't selected, they merely hide it. This means all those hidden charts continue to take up resources. Many of the widgets will undergo a redesign since they don't scale very well. After 11.2-BETA1 is out, the work on the reporting page will commence.

#6 Updated by Dru Lavigne about 2 years ago

  • Target version changed from 11.2-RC2 to 11.2-BETA3

#8 Updated by Dru Lavigne about 2 years ago

  • Has duplicate Bug #28868: Fix disk reporting graph added

#9 Updated by Erin Clark about 2 years ago

  • Target version changed from 11.2-BETA3 to 11.2-RC1

#11 Updated by Dru Lavigne almost 2 years ago

  • Target version changed from 11.2-RC1 to 11.2-RC2

#13 Updated by Erin Clark almost 2 years ago

  • Related to Bug #52151: Reporting - shows max 5 graphs added

#14 Updated by Dru Lavigne almost 2 years ago

  • Status changed from Screened to In Progress

#15 Updated by Damian Szidiropulosz almost 2 years ago

Fixed with PR 1434

#16 Updated by Dru Lavigne almost 2 years ago

  • Subject changed from DiskIO reporting page in NewUI cannot cope with a moderate sized pool to Improve Disk Reporting page in new UI
  • Status changed from In Progress to Ready for Testing
  • Needs Merging changed from Yes to No

#18 Updated by Dru Lavigne almost 2 years ago

  • Related to Bug #53748: SCSI target port graphs in new UI added

#19 Updated by Dru Lavigne almost 2 years ago

  • Subject changed from Improve Disk Reporting page in new UI to Add missing iSCSI target graph and improve Disk Reporting page in new UI

#21 Updated by Timothy Moore II almost 2 years ago

  • Needs Doc changed from Yes to No

Docs PR https://github.com/freenas/freenas-docs/pull/448 also resolved this issue.

#22 Updated by Dru Lavigne almost 2 years ago

  • Subject changed from Add missing iSCSI target graph and improve Disk Reporting page in new UI to Improve label of iSCSI target graph and improve Disk Reporting page in new UI

#24 Updated by Dru Lavigne almost 2 years ago

  • Status changed from Failed Testing to Done
  • Needs QA changed from Yes to No

#25 Updated by Dru Lavigne almost 2 years ago

  • Related to Bug #53566: Fix iSCSI target Reporting label in new UI added

Also available in: Atom PDF