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

Bug 1330476

Summary:	libgfapi:Setting need_lookup on wrong list
Product:	[Community] GlusterFS	Reporter:	Mohammed Rafi KC <rkavunga>
Component:	libgfapi	Assignee:	Mohammed Rafi KC <rkavunga>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Sudhir D <sdharane>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	bugs
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.8rc2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:
Clones:	1331264 (view as bug list)		Environment:
Last Closed:	2016-06-16 14:04:21 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:
Bug Blocks:	1331264

Description Mohammed Rafi KC 2016-04-26 10:17:51 UTC

Description of problem:

We are setting need_lookup on wrong list during readdirp cbk. we are using the previous list .
Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Vijay Bellur 2016-04-26 10:19:44 UTC

REVIEW: http://review.gluster.org/14073 (gfapi: set need_lookup flag on response list) posted (#1) for review on master by mohammed rafi  kc (rkavunga)

Comment 2 Vijay Bellur 2016-04-26 15:36:34 UTC

REVIEW: http://review.gluster.org/14073 (gfapi: set need_lookup flag on response list) posted (#2) for review on master by Niels de Vos (ndevos)

Comment 3 Vijay Bellur 2016-04-27 14:26:47 UTC

COMMIT: http://review.gluster.org/14073 committed in master by Niels de Vos (ndevos) 
------
commit 537557da59876536d33cd25a8ef485e7b5fe8849
Author: Mohammed Rafi KC <rkavunga>
Date:   Tue Apr 26 14:52:29 2016 +0530

    gfapi: set need_lookup flag on response list
    
    We set need_lookup flag for entries returned by readdirp
    to force lookup. Currently we are setting on the previously
    stored list, rather than response list returned by readdirp.
    
    This patch will iterate over current list returned by readdirp
    and will set need_lookup flag.
    
    Change-Id: Ibd6fcbc188f4c87f40ece7a9dcda27645401c240
    BUG: 1330476
    Signed-off-by: Mohammed Rafi KC <rkavunga>
    Reviewed-on: http://review.gluster.org/14073
    NetBSD-regression: NetBSD Build System <jenkins.org>
    Reviewed-by: Niels de Vos <ndevos>
    Reviewed-by: Rajesh Joseph <rjoseph>
    Smoke: Gluster Build System <jenkins.com>
    CentOS-regression: Gluster Build System <jenkins.com>

Comment 4 Niels de Vos 2016-06-16 14:04:21 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