Bug 1387328 - Log message shows error code as success even when rpc fails to connect
Summary: Log message shows error code as success even when rpc fails to connect
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Gluster Storage
Classification: Red Hat Storage
Component: rpc
Version: rhgs-3.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: RHGS 3.3.0
Assignee: Milind Changire
QA Contact: Vivek Das
URL:
Whiteboard:
Depends On: 1426032
Blocks: 1417147
TreeView+ depends on / blocked
 
Reported: 2016-10-20 16:09 UTC by Vivek Das
Modified: 2017-09-21 04:54 UTC (History)
5 users (show)

Fixed In Version: glusterfs-3.8.4-19
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1426032 (view as bug list)
Environment:
Last Closed: 2017-09-21 04:28:23 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2017:2774 0 normal SHIPPED_LIVE glusterfs bug fix and enhancement update 2017-09-21 08:16:29 UTC

Description Vivek Das 2016-10-20 16:09:50 UTC
Description of problem:

Log message shows error code as success even when rpc fails to connect

[2016-10-20 14:27:14.474001] E [MSGID: 104024] [glfs-mgmt.c:735:mgmt_rpc_notify] 0-glfs-mgmt: failed to connect with remote-host: 10.70.46.212 (Success)

Version-Release number of selected component (if applicable):
samba-client-4.4.6-2.el7rhgs.x86_64
glusterfs-client-xlators-3.8.4-2.el7rhgs.x86_64
Windows10

How reproducible:
Always

Steps to Reproduce:
1.Create a 4 node gluster cluster with ctdb
2.Enable volfile setup in smb.conf with unix+ socket (unix+/var/run/glusterd.socket tcp+<valid hostname>)
3.reload the smb config file (smbcontrol smbd reload-config)
4.kill glusterd in one of the server node
5.mount in windows using "net use" provide the ip of the server where glusterd is stopped

Actual results:
Success message on failure

Expected results:
No success message should come when error message is there

Additional info:

Comment 3 Raghavendra Talur 2017-02-03 07:30:37 UTC
This is a simple fix, just a change in gf_msg() call. errno should not be used in the log message, we should use a set error code to describe this error.

Comment 4 Milind Changire 2017-02-23 11:41:13 UTC
mainline patch: https://review.gluster.org/16730
moving to POST

Comment 6 Atin Mukherjee 2017-03-24 10:10:04 UTC
downstream patch : https://code.engineering.redhat.com/gerrit/#/c/101325/

Comment 8 Vivek Das 2017-04-24 09:44:41 UTC
Issue is fixed as did not find [E]rror code for the same.

Comment 10 errata-xmlrpc 2017-09-21 04:28:23 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:2774

Comment 11 errata-xmlrpc 2017-09-21 04:54:54 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:2774


Note You need to log in before you can comment on or make changes to this bug.