Bug 1230712

Summary:	[Backup]: Chown/chgrp for a directory does not get recorded as a MODIFY entry in the outfile
Product:	[Community] GlusterFS	Reporter:	Aravinda VK <avishwan>
Component:	glusterfind	Assignee:	Aravinda VK <avishwan>
Status:	CLOSED CURRENTRELEASE	QA Contact:	bugs <bugs>
Severity:	high	Docs Contact:
Priority:	high
Version:	3.7.0	CC:	amukherj, avishwan, bugs, gluster-bugs, khiremat, mchangir, sanandpa
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.7.2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1230127	Environment:
Last Closed:	2015-06-20 09:50:55 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:	1228597, 1230127
Bug Blocks:	1223636

Description Aravinda VK 2015-06-11 12:38:16 UTC

+++ This bug was initially created as a clone of Bug #1230127 +++

+++ This bug was initially created as a clone of Bug #1228597 +++

Description of problem:
Have a volume and glusterfind session(s) created for the same. When any metadata changes are done to any file, it gets captured in teh output file. However, when the owner/group of a directory is changed, there is no metion of the same in the output file.

Version-Release number of selected component (if applicable):
glusterfs-3.7.0-3.el6rhs.x86_64

How reproducible: Always


Steps to Reproduce:
1. Have a volume 2*2 'testvol' and glusterfind sessions 'sess1' and 'sess2'
2. Mount it over nfs/fuse and create 2 files and a directory 'dir1'
3. Change the permissions, owner, group of a file and run glusterfind pre and post
4. Verify that the file has a mention in the output file with the type MODIFY
5. Do the same for 'dir1' and execute glusterfind pre and post

Actual results:

Step5 generates an output file which does not have a mention of MODIFY op for the directory

Expected results:
Metadata changes to directory should get logged in the output file

--- Additional comment from Anand Avati on 2015-06-10 12:44:36 EDT ---

REVIEW: http://review.gluster.org/11158 (tools/glusterfind: Fix GFID to Path conversion for dir) posted (#2) for review on master by Aravinda VK (avishwan)

--- Additional comment from Anand Avati on 2015-06-11 02:42:00 EDT ---

REVIEW: http://review.gluster.org/11158 (tools/glusterfind: Fix GFID to Path conversion for dir) posted (#3) for review on master by Aravinda VK (avishwan)

Comment 1 Anand Avati 2015-06-11 12:39:38 UTC

REVIEW: http://review.gluster.org/11184 (tools/glusterfind: Fix GFID to Path conversion for dir) posted (#1) for review on release-3.7 by Aravinda VK (avishwan)

Comment 2 Anand Avati 2015-06-12 10:51:22 UTC

COMMIT: http://review.gluster.org/11184 committed in release-3.7 by Vijay Bellur (vbellur) 
------
commit 99633601f3719bd0bf5e5203ef844ebb087060e6
Author: Aravinda VK <avishwan>
Date:   Wed Jun 10 15:26:47 2015 +0530

    tools/glusterfind: Fix GFID to Path conversion for dir
    
    For Directories, GFID to Path conversion was fine, but wrong
    filter was used while updating to db.
    
    Change-Id: Id588adbb25ff5fd357f8cc2d5485bd2eb4a52b65
    Signed-off-by: Aravinda VK <avishwan>
    Reviewed-on: http://review.gluster.org/#/c/11158/
    BUG: 1230712
    Reviewed-on: http://review.gluster.org/11184
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 3 Niels de Vos 2015-06-20 09:50:55 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.2, please reopen this bug report.

glusterfs-3.7.2 has been announced on the Gluster Packaging mailinglist [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/2015-June/000006.html
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user