Bug 1402672

Summary:	Getting the warning message while erasing the gluster "glusterfs-server" package.
Product:	[Community] GlusterFS	Reporter:	Milind Changire <mchangir>
Component:	glusterfind	Assignee:	Milind Changire <mchangir>
Status:	CLOSED CURRENTRELEASE	QA Contact:	bugs <bugs>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	3.8	CC:	amukherj, ashah, avishwan, bsrirama, bugs, khiremat, rhs-bugs, storage-qa-internal
Target Milestone:	---
Target Release:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:	glusterfs-3.8.7	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1402369
Clones:	1407018 (view as bug list)		Environment:
Last Closed:	2016-12-14 07:13:24 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:	1402369
Bug Blocks:	1407018

Comment 1 Milind Changire 2016-12-08 06:17:20 UTC

Problem:
gluster volume delete causes the "post delete" glusterfind hook script to run. The "post delete" script mistakenly deletes the .keys directory under /var/lib/glusterd/glusterfind which may cause SSH keys to be deleted if there were glusterfind sessions created on the node.

Although the issue has become evident as part of erasing RPMs, the problem is not related to packaging.

Comment 2 Worker Ant 2016-12-08 06:18:42 UTC

REVIEW: http://review.gluster.org/16063 (tools/glusterfind: avoid deleting keys directory) posted (#1) for review on release-3.8 by Milind Changire (mchangir)

Comment 3 Worker Ant 2016-12-09 11:07:11 UTC

COMMIT: http://review.gluster.org/16063 committed in release-3.8 by Aravinda VK (avishwan) 
------
commit 1ebc8455641b82b8527b011de4a16f66b9a451e1
Author: Milind Changire <mchangir>
Date:   Wed Dec 7 17:27:02 2016 +0530

    tools/glusterfind: avoid deleting keys directory
    
    Problem:
    gluster volume delete mistakenly deletes the .keys directory under
    /var/lib/glusterd/glusterfind.
    
    Solution:
    Check for ".keys" directory and avoid deleting it.
    
    > BUG: 1402369
    > Reviewed-on: http://review.gluster.org/16052
    > Reviewed-by: Aravinda VK <avishwan>
    > Smoke: Gluster Build System <jenkins.org>
    > NetBSD-regression: NetBSD Build System <jenkins.org>
    > CentOS-regression: Gluster Build System <jenkins.org>
    (cherry picked from commit 2b4b928ed350286192b63b10b18f85c669b741f8)
    
    Change-Id: Ia595c8bf3f423c1ad5d6faa183a29598c07a11f9
    BUG: 1402672
    Signed-off-by: Milind Changire <mchangir>
    Reviewed-on: http://review.gluster.org/16063
    Smoke: Gluster Build System <jenkins.org>
    NetBSD-regression: NetBSD Build System <jenkins.org>
    CentOS-regression: Gluster Build System <jenkins.org>
    Reviewed-by: Aravinda VK <avishwan>

Comment 4 Niels de Vos 2016-12-14 07:13:24 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.7, please open a new bug report.

glusterfs-3.8.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] http://www.gluster.org/pipermail/packaging/2016-December/000226.html
[2] https://www.gluster.org/pipermail/gluster-users/

Comment 5 Worker Ant 2016-12-21 23:21:43 UTC

REVIEW: http://review.gluster.org/16251 (tools/glusterfind: avoid deleting keys directory) posted (#1) for review on release-3.8-fb by Kevin Vigor (kvigor)