Bug 1257394

Summary:	Provide more meaningful errors on peer probe and peer detach
Product:	[Community] GlusterFS	Reporter:	Brad Hubbard <bhubbard>
Component:	glusterd	Assignee:	Brad Hubbard <bhubbard>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Byreddy <bsrirama>
Severity:	low	Docs Contact:
Priority:	low
Version:	3.7.5	CC:	bugs, gluster-bugs, nlevinki, sasundar, vbellur
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	glusterfs-3.7.5	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1257149
Clones:	1260530 (view as bug list)		Environment:
Last Closed:	2015-10-14 10:29:41 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:	1257149
Bug Blocks:	1260530

Description Brad Hubbard 2015-08-27 02:13:06 UTC

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

Description of problem:

When a peer probe or peer detach fails glusterd currently reports "peer probe: failed: Probe returned with unknown errno 107". The process should be able to provide more information than that.

--- Additional comment from Brad Hubbard on 2015-08-26 22:03:17 EDT ---

http://review.gluster.org/#/c/12021/

Comment 1 Brad Hubbard 2015-09-07 05:48:26 UTC

http://review.gluster.org/#/c/12111/

Comment 2 Vijay Bellur 2015-09-07 07:51:25 UTC

REVIEW: http://review.gluster.org/12111 (glusterd: Return better error messages for probe and detach failures) posted (#2) for review on release-3.7 by Brad Hubbard (bhubbard)

Comment 3 Vijay Bellur 2015-09-08 08:57:07 UTC

COMMIT: http://review.gluster.org/12111 committed in release-3.7 by Vijay Bellur (vbellur) 
------
commit 4932409c92928ae1a6a2a158c4cd311b0c75973c
Author: Brad Hubbard <bhubbard>
Date:   Wed Aug 19 18:56:06 2015 +1000

    glusterd: Return better error messages for probe and detach failures
    
    We handle some specific errors and return good error messages for those, but for
    the default case where the error code is not recognised we just report "unknown
    errno". This patch attempts to at least return the output of strerror to provide
    more informative errors.
    
    Cherry picked from commit 4b5aec8da9be69da077e1fcc7e852d224517ecc0:
    >BUG: 1257149
    >Change-Id: I0027e74e41adac4ab0c0a929c6fff56878bf39c8
    >Signed-off-by: Brad Hubbard <bhubbard>
    >Reviewed-on: http://review.gluster.org/12021
    >Reviewed-by: Niels de Vos <ndevos>
    >Tested-by: NetBSD Build System <jenkins.org>
    >Reviewed-by: Atin Mukherjee <amukherj>
    
    BUG: 1257394
    Change-Id: I11e6c006c6b93a5c1b915f8c509f24123497301d
    Signed-off-by: Brad Hubbard <bhubbard>
    Reviewed-on: http://review.gluster.org/12111
    Reviewed-by: Niels de Vos <ndevos>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

Comment 4 Pranith Kumar K 2015-10-14 10:29:41 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-glusterfs-3.7.5, please open a new bug report.

glusterfs-glusterfs-3.7.5 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/gluster-users/2015-October/023968.html
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user

Comment 5 Pranith Kumar K 2015-10-14 10:38:31 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.5, please open a new bug report.

glusterfs-3.7.5 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/gluster-users/2015-October/023968.html
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user