stratis-storage / project

A holding place for issues that affect more than one repository in the project
4 stars 0 forks source link

Extend command for showing filesystem detail view #510

Open mulkieran opened 2 years ago

mulkieran commented 2 years ago

The filesystem list table view has six columns, column headers are:

This represents all the information about each filesystem that we put on the D-Bus, although some values are synthesized. One thing we have noticed it that users are surprised by our size numbers, because these are block layer sizes, not filesystem sizes. So, "Total" is the size of the thin device, and "Used" is the number of blocks used by the thin device. What we do not show and do not report on the D-Bus either, is the information obtained by statvfs on the xfs filesystem that is on the the thin device (this information is what df also shows, and that is what causes the user surprise).

A detail view for filesystems, analogous to that for pools, could show more information. We could also have the option of dropping one column from the table view, I'ld lean toward "Created". We could also modify the "Total / Used / Free" value in the table view to represent a summary triple that combines information from the two kinds of data, block level and filesystem level, if that seems more useful.

Part of the proposal would be to export two new values for the filesystem on the D-Bus, the xfs total and the xfs used. This should not be much work since all the infrastructure is in place.

drckeefe commented 2 years ago
  1. Created date - would be good.
  2. Showing both filesystem and block level could help.
  3. If the filesystem is full or not
  4. if the filesystem wants to grow (only when pool is full or in no overprovisioning mode)
  5. How about alerts? Can Stratis tell if the filesystem is in read-only mode or needs an fscheck?
  6. Could Stratis display the filesystem type? FS= XFS If the filesystem was not XFS it could either display / Warn
  7. (future) display if auto grow is turned off
mulkieran commented 2 years ago

At present we have no alerts on the filesystem, but we could. If we added Alerts, they should be in the table view, and we should definitely drop "Created". We might also want to drop "Total / Used / Free", as the Alerts would help the user know whether to look at the detail view, and the block-level value that we do show doesn't tell the whole story. The reason to drop columns is that the length of the display is long.

mulkieran commented 9 months ago

We might have reason to do this for snapshot support.

mulkieran commented 8 months ago

At present we now have the filesystem detail view added. But we still have the option to extend it to additional useful info.