Bug 1087676 - [SNAPSHOT]: gluster snapshot status timesout without displaying the output when snaps are >150
Summary: [SNAPSHOT]: gluster snapshot status timesout without displaying the output wh...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: mainline
Hardware: x86_64
OS: Linux
unspecified
urgent
Target Milestone: ---
Assignee: Sachin Pandit
QA Contact:
URL:
Whiteboard: SNAPSHOT
Depends On: 1082043
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-04-15 03:53 UTC by Sachin Pandit
Modified: 2014-11-11 08:30 UTC (History)
5 users (show)

Fixed In Version: glusterfs-3.6.0beta1
Doc Type: Bug Fix
Doc Text:
Clone Of: 1082043
Environment:
Last Closed: 2014-11-11 08:30:00 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Comment 1 Anand Avati 2014-04-15 03:54:42 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#7) for review on master by Sachin Pandit (spandit)

Comment 2 Anand Avati 2014-04-15 09:26:23 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#8) for review on master by Sachin Pandit (spandit)

Comment 3 Anand Avati 2014-04-23 02:47:13 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#9) for review on master by Sachin Pandit (spandit)

Comment 4 Anand Avati 2014-04-28 11:14:33 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#10) for review on master by Sachin Pandit (spandit)

Comment 5 Anand Avati 2014-04-29 07:29:12 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#11) for review on master by Sachin Pandit (spandit)

Comment 6 Anand Avati 2014-04-30 11:30:51 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#12) for review on master by Sachin Pandit (spandit)

Comment 7 Anand Avati 2014-04-30 14:09:05 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#13) for review on master by Sachin Pandit (spandit)

Comment 8 Anand Avati 2014-05-06 07:31:33 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#14) for review on master by Sachin Pandit (spandit)

Comment 9 Anand Avati 2014-05-06 09:58:54 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#15) for review on master by Sachin Pandit (spandit)

Comment 10 Anand Avati 2014-05-12 08:53:41 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status.) posted (#16) for review on master by Sachin Pandit (spandit)

Comment 11 Anand Avati 2014-05-12 09:38:13 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status) posted (#17) for review on master by Sachin Pandit (spandit)

Comment 12 Anand Avati 2014-05-20 06:41:49 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status) posted (#18) for review on master by Sachin Pandit (spandit)

Comment 13 Anand Avati 2014-05-20 08:28:25 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status) posted (#19) for review on master by Sachin Pandit (spandit)

Comment 14 Anand Avati 2014-05-29 08:44:25 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status) posted (#20) for review on master by Sachin Pandit (spandit)

Comment 15 Anand Avati 2014-06-02 09:25:49 UTC
REVIEW: http://review.gluster.org/7456 (glusterd/status : First fetch the snapcount and then send the rpc call for individual snapshots for snapshot status) posted (#21) for review on master by Sachin Pandit (spandit)

Comment 16 Anand Avati 2014-06-03 09:02:52 UTC
COMMIT: http://review.gluster.org/7456 committed in master by Krishnan Parthasarathi (kparthas) 
------
commit 31520d9334f9a40aa329d54651bca7e2c2e5d545
Author: Sachin Pandit <spandit>
Date:   Tue Apr 1 09:20:05 2014 +0530

    glusterd/status : First fetch the snapcount and then send the rpc call
    for individual snapshots for snapshot status
    
    Problem :
    
    Initially, we used to do all the calculation in the
    glusterd side, once all the information related to
    snap was fetched, it was aggregated into one
    dictionary and that was sent back to CLI. Problem
    with this approach was, when number of snapshots
    are very high then CLI will timeout.
    
    Solution:
    
    First fetch snapcount and snapname from glusterd,
    then make a individual calls using the snapname
    fetched. This will resolve the timeout problem.
    
    Change-Id: I32609b3898ed227c804dd4d8ee4516f081240756
    BUG: 1087676
    Signed-off-by: Sachin Pandit <spandit>
    Reviewed-on: http://review.gluster.org/7456
    Reviewed-by: Avra Sengupta <asengupt>
    Reviewed-by: Rajesh Joseph <rjoseph>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Krishnan Parthasarathi <kparthas>
    Tested-by: Krishnan Parthasarathi <kparthas>

Comment 17 Niels de Vos 2014-09-22 12:38:06 UTC
A beta release for GlusterFS 3.6.0 has been released. Please verify if the release solves this bug report for you. In case the glusterfs-3.6.0beta1 release does not have a resolution for this issue, leave a comment in this bug and move the status to ASSIGNED. If this release fixes the problem for you, leave a note and change the status to VERIFIED.

Packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update (possibly an "updates-testing" repository) infrastructure for your distribution.

[1] http://supercolony.gluster.org/pipermail/gluster-users/2014-September/018836.html
[2] http://supercolony.gluster.org/pipermail/gluster-users/

Comment 18 Niels de Vos 2014-11-11 08:30:00 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.6.1, please reopen this bug report.

glusterfs-3.6.1 has been announced [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://supercolony.gluster.org/pipermail/gluster-users/2014-November/019410.html
[2] http://supercolony.gluster.org/mailman/listinfo/gluster-users


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