Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1221481

Summary:	`ls' on a directory which has files with mismatching gfid's does not list anything
Product:	[Community] GlusterFS	Reporter:	Ravishankar N <ravishankar>
Component:	replicate	Assignee:	Ravishankar N <ravishankar>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	bugs, ravishankar, rkavunga, ssampat
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	glusterfs-3.8rc2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1218863
Clones:	1230242 (view as bug list)		Environment:
Last Closed:	2016-06-16 13:01:11 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:	1218863, 1223751
Bug Blocks:	1186580, 1230242

Description Ravishankar N 2015-05-14 07:03:25 UTC

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

Description of problem:
------------------------

Running `ls' a directory that has some files with mismatching gfid's on both bricks of a replica pair, does not list anything. All entries in the directory should be listed regardless of them being in split-brain.

Version-Release number of selected component (if applicable):
---------------------------------------------------------------
glusterfs-3.7.0alpha0-0.17.gited96153.el6.x86_64

How reproducible:
------------------
Always

Steps to Reproduce:
-------------------
1. On a distributed replicate volume (with self-heal-daemon turned off), bring down one brick in each replica pair and create a file on the mount point.
2. Bring all bricks up and then kill the other brick in each replica pair (different from the one that was brought down earlier)
3. Create a file with the same name on the mount point.
4. Run `ls' on the mount point.

Actual results:
----------------

`ls' does not list any entries.

Expected results:
------------------

`ls' should list all entries in the directory.

Additional info:

Comment 1 Anand Avati 2015-05-14 07:04:29 UTC

REVIEW: http://review.gluster.org/10776 (afr: allow readdir to proceed for directories in split-brain) posted (#1) for review on master by Ravishankar N (ravishankar)

Comment 2 Ravishankar N 2015-05-14 07:07:28 UTC

Changed the state to MODIFIED by mistake. Moving it back to POST.

Comment 3 Anand Avati 2015-05-14 08:53:12 UTC

REVIEW: http://review.gluster.org/10776 (afr: allow readdir to proceed for directories in split-brain) posted (#2) for review on master by Ravishankar N (ravishankar)

Comment 4 Anand Avati 2015-05-14 16:33:05 UTC

REVIEW: http://review.gluster.org/10776 (afr: allow readdir to proceed for directories in split-brain) posted (#3) for review on master by Ravishankar N (ravishankar)

Comment 5 Anand Avati 2015-05-16 04:07:22 UTC

REVIEW: http://review.gluster.org/10776 (afr: allow readdir to proceed for directories in split-brain) posted (#4) for review on master by Ravishankar N (ravishankar)

Comment 6 Anand Avati 2015-05-17 10:09:57 UTC

REVIEW: http://review.gluster.org/10776 (afr: allow readdir to proceed for directories in split-brain) posted (#5) for review on master by Vijay Bellur (vbellur)

Comment 7 Anand Avati 2015-05-19 08:57:53 UTC

REVIEW: http://review.gluster.org/10776 (afr: allow readdir to proceed for directories in split-brain) posted (#6) for review on master by Ravishankar N (ravishankar)

Comment 8 Anand Avati 2015-05-19 11:30:59 UTC

REVIEW: http://review.gluster.org/10776 (afr: allow readdir to proceed for directories in split-brain) posted (#7) for review on master by Ravishankar N (ravishankar)

Comment 9 Anand Avati 2015-05-28 09:20:38 UTC

COMMIT: http://review.gluster.org/10776 committed in master by Pranith Kumar Karampuri (pkarampu) 
------
commit 49b428433a03fcf709fdc8c08603b4cf02198e0a
Author: Ravishankar N <ravishankar>
Date:   Thu May 14 03:21:10 2015 +0530

    afr: allow readdir to proceed for directories in split-brain
    
    Problem:
    afr_read_txn() bails out if read_subvol==-1. This meant that for
    directories that were in entry split-brain, FOPS like readdir, access,
    stat etc were not allowed.
    
    Fix:
    Except for getxattr, all other FOPS are wound on the first up child
    of afr.
    
    Change-Id: Iacec8fbb1e75c4d2094baa304f62331c81a6f670
    BUG: 1221481
    Signed-off-by: Ravishankar N <ravishankar>
    Reviewed-on: http://review.gluster.org/10776
    Reviewed-by: Pranith Kumar Karampuri <pkarampu>
    Reviewed-by: Anuradha Talur <atalur>
    Tested-by: NetBSD Build System

Comment 10 Niels de Vos 2016-06-16 13:01:11 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