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

Bug 1217429

Summary:	geo-rep: add debug logs to master for slave ENTRY operation failures
Product:	[Community] GlusterFS	Reporter:	Milind Changire <mchangir>
Component:	geo-replication	Assignee:	Milind Changire <mchangir>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	3.7.0	CC:	amukherj, avishwan, bugs, gluster-bugs, vbellur
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.7.0beta2	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1207115	Environment:
Last Closed:	2015-05-15 17:16:18 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:

Description Milind Changire 2015-04-30 10:56:13 UTC

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

Description of problem:
Slave ENTRY operation failures cannot be traced due to lack of information on failing operations


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

How reproducible:
CREATE
Mount file-system on slave and create a file
Mount file-system on master the create the same file
Operation should fail on slave and there's no trace left behind about the failure

Similarly, RENAME and UNLINK operations also need to be tracked for failures on slave

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

--- Additional comment from Anand Avati on 2015-03-30 07:24:57 EDT ---

REVIEW: http://review.gluster.org/10048 (add logging on master for ENTRY operation errors from slave) posted (#1) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-03-30 07:26:53 EDT ---

REVIEW: http://review.gluster.org/10048 (geo-rep: add logging on master for ENTRY operation errors from slave) posted (#2) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-03-31 05:47:47 EDT ---

REVIEW: http://review.gluster.org/10048 (add logging on master for ENTRY operation errors from slave) posted (#3) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-03-31 05:50:59 EDT ---

REVIEW: http://review.gluster.org/10048 (geo-rep: add logging on master for ENTRY operation errors from slave) posted (#4) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-09 05:31:30 EDT ---

REVIEW: http://review.gluster.org/10048 (features/geo-rep: add logging on master for ENTRY operation errors from slave) posted (#5) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-10 02:30:15 EDT ---

REVIEW: http://review.gluster.org/10048 (features/geo-rep: add logging on master for ENTRY operation errors from slave) posted (#6) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-15 05:45:45 EDT ---

REVIEW: http://review.gluster.org/10048 (geo-rep: add logging on master for ENTRY operation errors from slave) posted (#7) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-24 03:18:46 EDT ---

REVIEW: http://review.gluster.org/10048 (geo-rep: add logging on master for ENTRY operation errors from slave) posted (#8) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-24 03:30:01 EDT ---

REVIEW: http://review.gluster.org/10048 (geo-rep: add logging on master for ENTRY operation errors from slave) posted (#9) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-27 05:59:13 EDT ---

REVIEW: http://review.gluster.org/10048 (geo-rep: add logging on master for ENTRY operation errors from slave) posted (#10) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-27 07:52:51 EDT ---

REVIEW: http://review.gluster.org/10048 (geo-rep: log ENTRY failures from slave on master) posted (#11) for review on master by Milind Changire (mchangir)

--- Additional comment from Anand Avati on 2015-04-28 01:05:20 EDT ---

COMMIT: http://review.gluster.org/10048 committed in master by Vijay Bellur (vbellur) 
------
commit 70a729e9751e45e266f7462443dcf2b6be3cecbe
Author: Milind Changire <mchangir>
Date:   Mon Mar 30 16:53:05 2015 +0530

    geo-rep: log ENTRY failures from slave on master
    
    ENTRY operations failures on slave left no trace for debugging purposes.
    This patch captures such failures on slave cluster and forwards them to
    the master and logs them. Failures of specific interest are the ones
    which return code EEXIST on the failing operations.
    
    Change-Id: Iecab876f16593c746d53f4b7ec2e0783367856bb
    BUG: 1207115
    Signed-off-by: Milind Changire <mchangir>
    Reviewed-on: http://review.gluster.org/10048
    Reviewed-by: Aravinda VK <avishwan>
    Tested-by: NetBSD Build System
    Tested-by: Gluster Build System <jenkins.com>

Comment 1 Anand Avati 2015-05-04 07:32:33 UTC

REVIEW: http://review.gluster.org/10517 (geo-rep: log ENTRY failures from slave on master) posted (#1) for review on release-3.7 by Milind Changire (mchangir)

Comment 2 Anand Avati 2015-05-05 10:13:30 UTC

REVIEW: http://review.gluster.org/10517 (geo-rep: log ENTRY failures from slave on master) posted (#3) for review on release-3.7 by Aravinda VK (avishwan)

Comment 3 Anand Avati 2015-05-06 08:03:29 UTC

REVIEW: http://review.gluster.org/10517 (geo-rep: log ENTRY failures from slave on master) posted (#4) for review on release-3.7 by Aravinda VK (avishwan)

Comment 4 Anand Avati 2015-05-06 16:07:24 UTC

COMMIT: http://review.gluster.org/10517 committed in release-3.7 by Vijay Bellur (vbellur) 
------
commit cfe717c3326ddb51b754bbc6370bac99dd02ec8c
Author: Milind Changire <mchangir>
Date:   Mon Mar 30 16:53:05 2015 +0530

    geo-rep: log ENTRY failures from slave on master
    
    ENTRY operations failures on slave left no trace for debugging purposes.
    This patch captures such failures on slave cluster and forwards them to
    the master and logs them. Failures of specific interest are the ones
    which return code EEXIST on the failing operations.
    
    Change-Id: Iecab876f16593c746d53f4b7ec2e0783367856bb
    Signed-off-by: Milind Changire <mchangir>
    BUG: 1217429
    Reviewed-on: http://review.gluster.org/10048
    Reviewed-by: Aravinda VK <avishwan>
    Reviewed-on: http://review.gluster.org/10517
    Tested-by: Gluster Build System <jenkins.com>
    Tested-by: NetBSD Build System

Comment 5 Niels de Vos 2015-05-15 17:16:18 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.0, please open a new bug report.

glusterfs-3.7.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://thread.gmane.org/gmane.comp.file-systems.gluster.devel/10939
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user