Bug 1408359 - `quota list` command displays 'N/A' when root of the volume is empty
Summary: `quota list` command displays 'N/A' when root of the volume is empty
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: GlusterFS
Classification: Community
Component: quota
Version: mainline
Hardware: x86_64
OS: Linux
low
low
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
: 1488386 1508661 (view as bug list)
Depends On:
Blocks: 1567864
TreeView+ depends on / blocked
 
Reported: 2016-12-23 04:49 UTC by Anoop C S
Modified: 2019-06-25 14:19 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
: 1567864 (view as bug list)
Environment:
Last Closed: 2019-06-25 14:19:42 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)
failing-test-case.t (325 bytes, application/x-shellscript)
2016-12-23 04:49 UTC, Anoop C S
no flags Details
passing-test-case.t (373 bytes, application/x-shellscript)
2016-12-23 04:50 UTC, Anoop C S
no flags Details

Description Anoop C S 2016-12-23 04:49:55 UTC
Created attachment 1234949 [details]
failing-test-case.t

Description of problem:
Even after setting a particular quota limit on '/' i.e, root of the volume, subsequent quota list command displays 'N/A' as usage limit for '/'.

How reproducible:
Always

Steps to Reproduce:
1. Apply https://review.gluster.org/#/c/12256/ on top of master branch.
2. Build and install GlusterFS
3. Run the attached test case named 'failing-test-case.t'

Actual results:
EXPECT_WITHIN $MARKER_UPDATE_TIMEOUT "10.0MB" quota_list_field "/" 5
not ok 6 Got "N/A" instead of "10.0MB", LINENUM:17

Expected results:
Tests should complete successfully.

Additional info:
As of now root of the volume is non-empty as we were having .trashcan directory created by default. But with the patch mentioned in 'Steps to Reproduce' this behaviour is being targeted to change by 3.10. See https://github.com/gluster/glusterfs/issues/65 for more details.

A modified test case named 'passing-test-case.t' is also attached which runs successfully by creating a test file at root of the volume.

Comment 1 Anoop C S 2016-12-23 04:50:39 UTC
Created attachment 1234950 [details]
passing-test-case.t

Comment 2 Sanoj Unnikrishnan 2017-07-27 13:33:24 UTC
THe issue is that the root path does not have the size xattr and hence in both 
print_quota_list_from_mountdir and print_quota_list_from_quotad, we print N/A (print_quota_list_empty) if the size xattr is not found. A stat on the root path does not still heal the size xattr. Needs to RCA fruther.

Comment 3 Sanoj Unnikrishnan 2017-11-03 09:59:55 UTC
*** Bug 1488386 has been marked as a duplicate of this bug. ***

Comment 4 Sanoj Unnikrishnan 2017-11-10 05:42:59 UTC
*** Bug 1508661 has been marked as a duplicate of this bug. ***

Comment 5 Amar Tumballi 2019-06-25 14:19:42 UTC
Not planning to work on this in near future. Will revisit when we get a chance.


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