Bug 1242329 - [Quota] : Inode quota spurious failure
Summary: [Quota] : Inode quota spurious failure
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: quota
Version: 3.7.2
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard: quota
Depends On: 1229297
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-07-13 05:32 UTC by Vijaikumar Mallikarjuna
Modified: 2016-05-11 22:47 UTC (History)
5 users (show)

Fixed In Version: glusterfs-3.7.3
Doc Type: Bug Fix
Doc Text:
Clone Of: 1229297
Environment:
Last Closed: 2015-07-30 09:48:45 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Vijaikumar Mallikarjuna 2015-07-13 05:32:52 UTC
+++ This bug was initially created as a clone of Bug #1229297 +++

I did some analysis on this and found out below mentioned
log during the case of failure.

[2015-06-08 03:45:21.317352] I [MSGID: 115071] [server-rpc-fops.c:1591:server_crea
te_cbk] 0-patchy-server: 203: CREATE /test_dir/test1.txt (72fa9c7d-d4cd-474e-baae-
28dde9b15990/test1.txt) ==> (Disk quota exceeded) [Disk quota exceeded]

The test case which fails is "TEST_IN_LOOP touch $M0/test_dir/test$i.txt",
while creating a file "test1" (i.e i=1). This should not fail as "test1" is
the first file being created in the directory where quota limit is set.
The only possible explanation is that before creating the file "test1",
we delete all the files present in that directory, and that information
has not been updated by marker. Because of which the new file creation
fails thinking the quota limit is still exceeded.

The test case basically validates both the types of quota (size and inode).
The reason for this spurious failure might be because of the delay in
updating the information my marker. I will verify the same by inducing
some delay before the execution of above mentioned testcase.

--- Additional comment from Anand Avati on 2015-06-08 08:13:40 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#1) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-09 01:08:33 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#2) for review on master by Vijay Bellur (vbellur)

--- Additional comment from Anand Avati on 2015-06-09 01:38:00 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#3) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-09 02:54:56 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#4) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-09 04:16:07 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#5) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-12 04:36:18 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#6) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-15 01:45:10 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#7) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-23 06:33:27 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#8) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-25 05:58:33 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#10) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-06-30 05:30:09 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#11) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-07-09 01:46:15 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#12) for review on master by Sachin Pandit (spandit)

--- Additional comment from Anand Avati on 2015-07-10 05:23:57 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#13) for review on master by Vijaikumar Mallikarjuna (vmallika)

--- Additional comment from Anand Avati on 2015-07-11 23:25:43 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#14) for review on master by Vijaikumar Mallikarjuna (vmallika)

--- Additional comment from Anand Avati on 2015-07-11 23:32:19 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#15) for review on master by Vijaikumar Mallikarjuna (vmallika)

--- Additional comment from Anand Avati on 2015-07-12 11:44:22 EDT ---

REVIEW: http://review.gluster.org/11125 (features/quota : Fix spurious failure) posted (#16) for review on master by Vijaikumar Mallikarjuna (vmallika)

--- Additional comment from Anand Avati on 2015-07-13 00:34:59 EDT ---

COMMIT: http://review.gluster.org/11125 committed in master by Krishnan Parthasarathi (kparthas) 
------
commit 46d4d7ade012f86044135dd4af3101f851484cda
Author: vmallika <vmallika>
Date:   Fri Jul 10 14:37:50 2015 +0530

    features/quota : Fix spurious failure
    
    Problem : Basically, in this test case a file is created
    which exceeds the quota limit. Once the limit is reached
    that file will be deleted. At the same moment we are
    testing inode-quota. It can so happen that before the
    marker updates the information related to deletion of
    file, a new file creation operation comes and sees that
    quota limit is still exceeded.
    
    Solution : Inducing a check to see if marker updation
    completed successfully.
    
    Updated all the test case which has the similar
    machanism and also moved the "usage" function
    to a common place "volume.rc"
    
    Change-Id: I36ddbc5ebbf1b74c9d326a0d1d5f3b32f20a906a
    BUG: 1229297
    Signed-off-by: Sachin Pandit <spandit>
    Signed-off-by: vmallika <vmallika>
    Reviewed-on: http://review.gluster.org/11125
    Tested-by: NetBSD Build System <jenkins.org>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Krishnan Parthasarathi <kparthas>

Comment 1 Anand Avati 2015-07-15 10:03:51 UTC
REVIEW: http://review.gluster.org/11674 (quota: fix quota test case) posted (#1) for review on release-3.7 by Vijaikumar Mallikarjuna (vmallika)

Comment 2 Anand Avati 2015-07-15 12:09:50 UTC
REVIEW: http://review.gluster.org/11674 (quota: fix quota test case) posted (#2) for review on release-3.7 by Vijaikumar Mallikarjuna (vmallika)

Comment 3 Kaushal 2015-07-30 09:48:45 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.3, please open a new bug report.

glusterfs-3.7.3 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/12078
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 4 Kaushal 2015-07-30 09:50:52 UTC
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.7.3, please open a new bug report.

glusterfs-3.7.3 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/12078
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user


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