Bug 1291546

Summary: bitrot: bitrot scrub status command should display the correct value of total number of scrubbed, unsigned file
Product: [Community] GlusterFS Reporter: Gaurav Kumar Garg <ggarg>
Component: bitrotAssignee: Gaurav Kumar Garg <ggarg>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact: bugs <bugs>
Priority: unspecified    
Version: 3.7.0CC: bugs, smohan
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: glusterfs-3.7.7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1285989 Environment:
Last Closed: 2016-04-19 07:51:02 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: 1285989, 1329211    
Bug Blocks:    

Description Gaurav Kumar Garg 2015-12-15 06:13:56 UTC
+++ This bug was initially created as a clone of Bug #1285989 +++

Description of problem:

if user execute bitrot scrub status command then it should display correct value of total number of scrubbed file, total number of unsigned file, and last scrub time.

--- Additional comment from Vijay Bellur on 2015-11-27 02:22:00 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#1) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-11-27 10:57:21 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#2) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-03 22:45:33 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#4) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-04 04:43:14 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#5) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-09 04:15:15 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#6) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-09 04:46:28 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#7) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-09 06:38:21 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#8) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-10 02:18:31 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#9) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-10 03:45:07 EST ---

REVIEW: http://review.gluster.org/12776 (bitrot: getting correct value of scrub stat's) posted (#10) for review on master by Gaurav Kumar Garg (ggarg)

--- Additional comment from Vijay Bellur on 2015-12-14 22:08:43 EST ---

COMMIT: http://review.gluster.org/12776 committed in master by Venky Shankar (vshankar) 
------
commit 22827d51c232c44a8f5ac003529d907d93baf7b0
Author: Gaurav Kumar Garg <garg.gaurav52>
Date:   Wed Nov 25 17:38:43 2015 +0530

    bitrot: getting correct value of scrub stat's
    
    When user execute bitrot scrub status command then gluster
    is not giving correct value of Number of Scrubbed files,
    Number of Unsigned files, Last completed scrub time,
    Duration of last scrub.
    
    With this patch scrub status will give correct value for
    all the above fields.
    
    Change-Id: Ic966f76d22db5b0c889e6386a1c2219afbda1f49
    BUG: 1285989
    Signed-off-by: Gaurav Kumar Garg <ggarg>
    Signed-off-by: Kotresh HR <khiremat>
    Reviewed-on: http://review.gluster.org/12776
    Tested-by: NetBSD Build System <jenkins.org>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Venky Shankar <vshankar>

Comment 1 Vijay Bellur 2015-12-15 06:15:57 UTC
REVIEW: http://review.gluster.org/12966 (bitrot: getting correct value of scrub stat's) posted (#1) for review on release-3.7 by Gaurav Kumar Garg (ggarg)

Comment 2 Vijay Bellur 2015-12-18 03:29:06 UTC
COMMIT: http://review.gluster.org/12966 committed in release-3.7 by Venky Shankar (vshankar) 
------
commit 530533fe44f08f30d1230c65db0054be7c6d3281
Author: Gaurav Kumar Garg <garg.gaurav52>
Date:   Wed Nov 25 17:38:43 2015 +0530

    bitrot: getting correct value of scrub stat's
    
    This patch is backport of: http://review.gluster.org/#/c/12776/
    
    When user execute bitrot scrub status command then gluster
    is not giving correct value of Number of Scrubbed files,
    Number of Unsigned files, Last completed scrub time,
    Duration of last scrub.
    
    With this patch scrub status will give correct value for
    all the above fields.
    
      >> Change-Id: Ic966f76d22db5b0c889e6386a1c2219afbda1f49
      >> BUG: 1285989
      >> Signed-off-by: Gaurav Kumar Garg <ggarg>
      >> Signed-off-by: Kotresh HR <khiremat>
      >> Reviewed-on: http://review.gluster.org/12776
      >> Tested-by: NetBSD Build System <jenkins.org>
      >> Tested-by: Gluster Build System <jenkins.com>
      >> Reviewed-by: Venky Shankar <vshankar>
    
    Change-Id: Ic966f76d22db5b0c889e6386a1c2219afbda1f49
    BUG: 1291546
    Signed-off-by: Gaurav Kumar Garg <ggarg>
    
    
    (cherry picked from commit 22827d51c232c44a8f5ac003529d907d93baf7b0)
    
    Change-Id: Icef24cce35c8d54ffdfa5282491338318e78780b
    Reviewed-on: http://review.gluster.org/12966
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System <jenkins.org>
    Reviewed-by: Venky Shankar <vshankar>

Comment 3 Vijay Bellur 2016-01-12 06:12:55 UTC
REVIEW: http://review.gluster.org/13223 (glusterd/cli: mask out inaccurate scrub statistics) posted (#1) for review on release-3.7 by Venky Shankar (vshankar)

Comment 4 Vijay Bellur 2016-01-26 14:18:55 UTC
REVIEW: http://review.gluster.org/13223 (glusterd/cli: mask out inaccurate scrub statistics) posted (#2) for review on release-3.7 by Venky Shankar (vshankar)

Comment 5 Vijay Bellur 2016-01-27 11:50:02 UTC
COMMIT: http://review.gluster.org/13223 committed in release-3.7 by Atin Mukherjee (amukherj) 
------
commit 1643209297664462ffabc04a4fe667f2fb9be741
Author: Venky Shankar <vshankar>
Date:   Tue Dec 29 10:54:55 2015 +0530

    glusterd/cli: mask out inaccurate scrub statistics
    
            Backport of http://review.gluster.org/13105
    
    Some of the scrub statistics are inaccurate at the moment and would
    successfully fool users at times. It's best to not display these
    stats and stick to what is accurate.
    
    Note that the change is only in the CLI part - glusterd (and brick)
    supplies _all_ available scrub statistics, but CLI just selectively
    sticks to what needs to be displayed.
    
    Change-Id: Ia99f505a2cb10d5406cd37035d082e593f7e9a8c
    BUG: 1291546
    Signed-off-by: Venky Shankar <vshankar>
    Reviewed-on: http://review.gluster.org/13223
    Smoke: Gluster Build System <jenkins.com>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.com>
    Reviewed-by: Atin Mukherjee <amukherj>

Comment 6 Kaushal 2016-04-19 07:51:02 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.7, please open a new bug report.

glusterfs-3.7.7 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] https://www.gluster.org/pipermail/gluster-users/2016-February/025292.html
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user