Project

General

Profile

Bug #34882

Use tree view for boot pool status page in new UI

Added by Warren Block over 2 years ago. Updated about 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Vaibhav Chauhan
Category:
GUI (new)
Target version:
Severity:
New
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

System/Boot/Boot Pool Status shows the boot pool both by name "freenas-boot" and by device ID (/dev/ada0p2). These are the same device, and only one should be shown.

bootpoolstatus.png (106 KB) bootpoolstatus.png Warren Block, 06/13/2018 07:06 AM
18427

History

#1 Updated by Erin Clark over 2 years ago

  • Assignee changed from Erin Clark to Vaibhav Chauhan

#2 Updated by William Grzybowski over 2 years ago

I dont agree with the description. Both should be shown but type should not be shown for freenas-boot.

This should mimic zpool status just like legacy UI. Ideally as a tree view.

Remember it could also be a mirror.

#3 Updated by Warren Block over 2 years ago

  • Subject changed from Boot pool is shown twice in Boot Pool Status to Boot pool drives are shown twice in Boot Pool Status

The boot pool in the example was a single device, da0. freenas-boot (a GPT label) and da0p2 refer to the same thing, the single drive (well, partition) in the boot pool. There are multiple paths to that device. The way it is shown implies that these are actually two different devices in the pool. Showing this to the user as two line items is going to cause confusion. Showing one line per physical device would be better. The device paths are unique and are probably the thing to display. We could add a column for GPT labels if that will be helpful to the user.

If you mean we should have one entry for the vdev with additional lines for each drive in that vdev, I can agree, provided we have a clear way to show that the drives are a part of that vdev and don't look like additional vdevs. Maybe indenting the child drives? Or a menu entry to show drives?

I have edited the description.

#4 Updated by William Grzybowski over 2 years ago

Well, I am saying it should have the same tree view output as regular pool status we have today.

#5 Updated by Dru Lavigne over 2 years ago

  • Target version changed from 11.2-BETA1 to 11.2-BETA2

#6 Updated by Vaibhav Chauhan about 2 years ago

  • Status changed from Unscreened to Screened

#7 Updated by Vaibhav Chauhan about 2 years ago

  • Status changed from Screened to In Progress

#8 Updated by Erin Clark about 2 years ago

  • Status changed from In Progress to Ready for Testing

#9 Updated by Dru Lavigne about 2 years ago

  • Subject changed from Boot pool drives are shown twice in Boot Pool Status to Use tree view for boot pool status page in new UI
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#10 Updated by Bonnie Follweiler about 2 years ago

  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No

#11 Updated by Dru Lavigne about 2 years ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF