Bug 1225465 - [Backup]: Glusterfind session entry persists even after volume is deleted
Summary: [Backup]: Glusterfind session entry persists even after volume is deleted
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterfind
Version: mainline
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
Assignee: Aravinda VK
QA Contact: bugs@gluster.org
URL:
Whiteboard:
Depends On:
Blocks: 1223636 1224880 1225551 1256307
TreeView+ depends on / blocked
 
Reported: 2015-05-27 12:41 UTC by Aravinda VK
Modified: 2016-06-16 13:05 UTC (History)
3 users (show)

Fixed In Version: glusterfs-3.8rc2
Doc Type: Bug Fix
Doc Text:
Clone Of: 1224064
: 1225551 1256307 (view as bug list)
Environment:
Last Closed: 2016-06-16 13:05:25 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Aravinda VK 2015-05-27 12:41:01 UTC
+++ This bug was initially created as a clone of Bug #1224064 +++

Description of problem:
When a volume is created and a corresponding glusterfind session, glusterfind list displays the entry of the session name and volume name. After deletion of the volume, the session entry should get removed from the display output of glusterfind list. Glusterfind list should always have only the list of active glusterfind sessions that are present in that cluster.


Version-Release number of selected component (if applicable):


How reproducible:
Always


Additional info:


[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# gluster v info
 
Volume Name: nash
Type: Distributed-Replicate
Volume ID: ef2333ce-e513-43df-8306-fec77cc479b4
Status: Started
Number of Bricks: 2 x 2 = 4
Transport-type: tcp
Bricks:
Brick1: 10.70.43.140:/rhs/thinbrick1/nash/dd
Brick2: 10.70.42.75:/rhs/thinbrick1/dd
Brick3: 10.70.43.140:/rhs/thinbrick2/nash/dd
Brick4: 10.70.42.75:/rhs/thinbrick2/dd
Options Reconfigured:
performance.readdir-ahead: on
 
Volume Name: vol1
Type: Distributed-Replicate
Volume ID: 44f06391-1635-4897-98c2-848e5ae92640
Status: Started
Number of Bricks: 2 x 2 = 4
Transport-type: tcp
Bricks:
Brick1: 10.70.43.140:/rhs/brick1/dd
Brick2: 10.70.42.75:/rhs/brick1/dd
Brick3: 10.70.43.140:/rhs/brick2/dd
Brick4: 10.70.42.75:/rhs/brick2/dd
Options Reconfigured:
changelog.changelog: on
storage.build-pgfid: on
performance.readdir-ahead: on
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# glusterfind list
SESSION                   VOLUME                    SESSION TIME             
---------------------------------------------------------------------------
svol1                     vol1                      2015-05-22 17:02:01      
snash                     nash                      2015-05-22 17:02:58      
sess1                     ozone                     Session Corrupted        
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# gluster v stop nash
Stopping volume will make its data inaccessible. Do you want to continue? (y/n) y
volume stop: nash: success
[root@dhcp43-140 ~]# gluster v delete nash
Deleting volume will erase all information about the volume. Do you want to continue? (y/n) y
volume delete: nash: success
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# gluster v info
 
Volume Name: vol1
Type: Distributed-Replicate
Volume ID: 44f06391-1635-4897-98c2-848e5ae92640
Status: Stopped
Number of Bricks: 2 x 2 = 4
Transport-type: tcp
Bricks:
Brick1: 10.70.43.140:/rhs/brick1/dd
Brick2: 10.70.42.75:/rhs/brick1/dd
Brick3: 10.70.43.140:/rhs/brick2/dd
Brick4: 10.70.42.75:/rhs/brick2/dd
Options Reconfigured:
changelog.changelog: on
storage.build-pgfid: on
performance.readdir-ahead: on
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# glusterfind list
SESSION                   VOLUME                    SESSION TIME             
---------------------------------------------------------------------------
svol1                     vol1                      2015-05-22 17:02:01      
snash                     nash                      2015-05-22 17:02:58      
sess1                     ozone                     Session Corrupted        
[root@dhcp43-140 ~]# 
[root@dhcp43-140 ~]# 

[root@dhcp43-140 ~]#


--- Additional comment from Aravinda VK on 2015-05-27 04:02:07 EDT ---

glusterfind is a independent tool which does not know when volume is deleted. We need to run `glusterfind delete` command before volume delete.

Comment 1 Anand Avati 2015-05-27 12:59:10 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#1) for review on master by Aravinda VK (avishwan)

Comment 2 Anand Avati 2015-05-29 07:34:23 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#2) for review on master by Aravinda VK (avishwan)

Comment 3 Anand Avati 2015-06-02 07:22:48 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#3) for review on master by Aravinda VK (avishwan)

Comment 4 Anand Avati 2015-06-04 07:14:31 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#4) for review on master by Aravinda VK (avishwan)

Comment 5 Anand Avati 2015-06-04 08:06:13 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#5) for review on master by Aravinda VK (avishwan)

Comment 6 Anand Avati 2015-06-04 10:16:52 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#6) for review on master by Aravinda VK (avishwan)

Comment 7 Anand Avati 2015-06-09 08:36:25 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#8) for review on master by Aravinda VK (avishwan)

Comment 8 Anand Avati 2015-06-10 16:43:38 UTC
REVIEW: http://review.gluster.org/10944 (tools/glusterfind: Cleanup glusterfind dir after a volume delete) posted (#9) for review on master by Aravinda VK (avishwan)

Comment 9 Anand Avati 2015-06-12 10:01:21 UTC
COMMIT: http://review.gluster.org/10944 committed in master by Vijay Bellur (vbellur) 
------
commit d28b226131d420070fa5cee921a4ad0be9d6446a
Author: Aravinda VK <avishwan>
Date:   Wed May 27 18:05:35 2015 +0530

    tools/glusterfind: Cleanup glusterfind dir after a volume delete
    
    If `glusterfind delete` command was not run before volume delete, stale
    session directories exists in /var/lib/glusterd/glusterfind directories.
    
    Also shows these sessions in `glusterfind list`
    
    When Volume is deleted, Post hook will be run which cleans up the stale
    session directories
    
    BUG: 1225465
    Change-Id: I54c46c30313e92c1bb4cb07918ed2029b375462c
    Signed-off-by: Aravinda VK <avishwan>
    Reviewed-on: http://review.gluster.org/10944
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Kotresh HR <khiremat>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 10 Anand Avati 2015-06-17 22:25:24 UTC
REVIEW: http://review.gluster.org/11298 (rpm: include required directory for glusterfind) posted (#1) for review on master by Niels de Vos (ndevos)

Comment 11 Anand Avati 2015-06-18 08:51:26 UTC
REVIEW: http://review.gluster.org/11298 (rpm: include required directory for glusterfind) posted (#2) for review on master by Niels de Vos (ndevos)

Comment 12 Anand Avati 2015-08-18 09:31:32 UTC
REVIEW: http://review.gluster.org/11298 (rpm: include required directory for glusterfind) posted (#3) for review on master by Aravinda VK (avishwan)

Comment 13 Anand Avati 2015-08-19 05:21:43 UTC
REVIEW: http://review.gluster.org/11298 (rpm: include required directory for glusterfind) posted (#4) for review on master by Aravinda VK (avishwan)

Comment 14 Anand Avati 2015-08-19 10:33:17 UTC
COMMIT: http://review.gluster.org/11298 committed in master by Kaleb KEITHLEY (kkeithle) 
------
commit 454bd09b8befc27552591855a8d02a0ad19877d9
Author: Niels de Vos <ndevos>
Date:   Thu Jun 18 00:21:59 2015 +0200

    rpm: include required directory for glusterfind
    
    The directory was marked as %ghost, which causes the following
    installation failure:
    
      Error unpacking rpm package glusterfs-server-3.8dev-0.446.git45e13fe.el7.centos.x86_64
      error: unpacking of archive failed on file /var/lib/glusterd/hooks/1/delete/post/S57glusterfind-delete-post.py;5581f20e: cpio: open
    
    Also, *all* Python files should be part of the RPM package. This
    includes generated .pyc and .pyo files.
    
    BUG: 1225465
    Change-Id: Iee74905b101912c4a845257742c470c3fe42ce2a
    Signed-off-by: Niels de Vos <ndevos>
    Signed-off-by: Aravinda VK <avishwan>
    Reviewed-on: http://review.gluster.org/11298
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Kaleb KEITHLEY <kkeithle>

Comment 15 Anand Avati 2015-08-24 10:45:02 UTC
REVIEW: http://review.gluster.org/12000 (rpm: include required directory for glusterfind) posted (#1) for review on release-3.7 by Aravinda VK (avishwan)

Comment 16 Nagaprasad Sathyanarayana 2015-10-25 15:00:08 UTC
Fix for this BZ is already present in a GlusterFS release. You can find clone of this BZ, fixed in a GlusterFS release and closed. Hence closing this mainline BZ as well.

Comment 17 Niels de Vos 2016-06-16 13:05:25 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.8.0, please open a new bug report.

glusterfs-3.8.0 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://blog.gluster.org/2016/06/glusterfs-3-8-released/
[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.