Bug 1101382 - [RFE] glusterd log could also add hostname or ip along with host's UUID
Summary: [RFE] glusterd log could also add hostname or ip along with host's UUID
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: glusterd
Version: mainline
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Gaurav Kumar Garg
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1101395
TreeView+ depends on / blocked
 
Reported: 2014-05-27 04:29 UTC by SATHEESARAN
Modified: 2016-06-05 23:37 UTC (History)
4 users (show)

Fixed In Version: glusterfs-3.7.0
Clone Of:
: 1101395 (view as bug list)
Environment:
Last Closed: 2015-05-14 17:25:48 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description SATHEESARAN 2014-05-27 04:29:12 UTC
Description of problem:
-----------------------
glusterd logs identifies host using host's UUID.
Need to identify the host after translating it in to corresponding host.
It would be helpful, if the logs references UUID along with hostname or IP

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

How reproducible:
-----------------
Not Applicable as this is just a enhancement

Steps to Reproduce:
-------------------
1. Setup the gluster cluster with atleast 2 nodes
2. stop glusterd in one of the node
3. Look for glusterd logs on another node

Actual results:
---------------
glusterd log shows that peer's UUID as follows,
[2014-05-27 07:20:09.033054] I [MSGID: 106004] [glusterd-handler.c:4290:__glusterd_peer_rpc_notify] 0-management: Peer 36efbc95-bee4-4b44-ab7e-1a4bf1903cf7, in Peer in Cluster state, has disconnected from glusterd.

Expected results:
-----------------
UUID along with hostname or IP could help anyone reading it

Comment 1 Anand Avati 2014-11-17 07:30:31 UTC
REVIEW: http://review.gluster.org/9136 (glusterd: add ip address along with host's UUID in glusterd log message) posted (#1) for review on master by Gaurav Kumar Garg (ggarg)

Comment 2 Anand Avati 2014-11-17 08:54:19 UTC
REVIEW: http://review.gluster.org/9136 (glusterd: add ip address along with host's UUID in glusterd log message) posted (#2) for review on master by Gaurav Kumar Garg (ggarg)

Comment 3 Anand Avati 2014-11-17 09:22:10 UTC
REVIEW: http://review.gluster.org/9136 (glusterd: Add hostname/ip-address along with host's UUID in glusterd log message) posted (#3) for review on master by Gaurav Kumar Garg (ggarg)

Comment 4 Anand Avati 2014-11-17 10:02:33 UTC
REVIEW: http://review.gluster.org/9136 (glusterd: Add hostname/ip-address along with host's UUID in glusterd log message) posted (#4) for review on master by Gaurav Kumar Garg (ggarg)

Comment 5 Anand Avati 2014-11-21 05:20:49 UTC
COMMIT: http://review.gluster.org/9136 committed in master by Kaushal M (kaushal) 
------
commit 7d0bfc90027c824e6d49cba7c3c3dc818f9c365b
Author: GauravKumarGarg <ggarg>
Date:   Mon Nov 17 12:50:49 2014 +0530

    glusterd: Add hostname/ip-address along with host's UUID in glusterd log message
    
    Previously when host disconnected from cluster then glusterd logs
    identifies host using host's UUID.
    
    Now with this fix, UUID along with host's ip will be present in
    glusterd log message when one of the peer disconnected from cluster.
    So it will enhancement better readability of user from log file.
    
    Change-Id: I3b7eaf1b1a8963ef2096e67a78cf69f67d5d5166
    BUG: 1101382
    Signed-off-by: Gaurav Kumar Garg <ggarg>
    Reviewed-on: http://review.gluster.org/9136
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Krishnan Parthasarathi <kparthas>
    Reviewed-by: Kaushal M <kaushal>

Comment 6 Niels de Vos 2015-05-14 17:25:48 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

Comment 7 Niels de Vos 2015-05-14 17:35:26 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

Comment 8 Niels de Vos 2015-05-14 17:37:48 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

Comment 9 Niels de Vos 2015-05-14 17:42: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


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