Bug 1408359

Summary: `quota list` command displays 'N/A' when root of the volume is empty
Product: [Community] GlusterFS Reporter: Anoop C S <anoopcs>
Component: quotaAssignee: bugs <bugs>
Status: CLOSED WONTFIX QA Contact:
Severity: low Docs Contact:
Priority: low    
Version: mainlineCC: atumball, bugs, jthottan, vnosov, zhou.lei56
Target Milestone: ---Keywords: Triaged
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1567864 (view as bug list) Environment:
Last Closed: 2019-06-25 14:19:42 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1567864    
Attachments:
Description Flags
failing-test-case.t
none
passing-test-case.t none

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.