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

Description of problem:

'gluster volume status <vol>' command doesn't display host name for NFS and SHD services, instead it displays the UUID of the peers.

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

glusterfs 3.4.0alpha2 built on Mar  6 2013 23:54:05

How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:

[root@localhost ~]# gluster volume status r1
Status of volume: r1
Gluster process                                         Port    Online  Pid
------------------------------------------------------------------------------
Brick 10.70.37.211:/tmp/r1a                             49163   Y       28822
Brick 10.70.37.216:/tmp/r1b                             49156   Y       9345
NFS Server on localhost                                 38467   Y       28834
Self-heal Daemon on localhost                           N/A     Y       28846
NFS Server on 24b83353-b2d7-4563-b4ea-7ebfacfcf3bb      38467   Y       9487
Self-heal Daemon on 24b83353-b2d7-4563-b4ea-7ebfacfcf3b
b                                                       N/A     Y       9499
NFS Server on 19f601e2-e9be-41a0-8b1c-0f4334cf3e44      38467   Y       9358
Self-heal Daemon on 19f601e2-e9be-41a0-8b1c-0f4334cf3e4
4                                                       N/A     Y       9371
 
There are no active volume tasks


Expected results:


Additional info:

--- Additional comment from Kanagaraj on 2013-03-26 17:18:58 IST ---

[root@localhost ~]# gluster volume status r1 --xml
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<cliOutput>
  <opRet>0</opRet>
  <opErrno>115</opErrno>
  <opErrstr/>
  <volStatus>
    <volumes>
      <volume>
        <volName>r1</volName>
        <nodeCount>8</nodeCount>
        <node>
          <hostname>10.70.37.211</hostname>
          <path>/tmp/r1a</path>
          <status>1</status>
          <port>49163</port>
          <pid>28822</pid>
        </node>
        <node>
          <hostname>10.70.37.216</hostname>
          <path>/tmp/r1b</path>
          <status>1</status>
          <port>49156</port>
          <pid>9345</pid>
        </node>
        <node>
          <hostname>NFS Server</hostname>
          <path>localhost</path>
          <status>1</status>
          <port>38467</port>
          <pid>28834</pid>
        </node>
        <node>
          <hostname>Self-heal Daemon</hostname>
          <path>localhost</path>
          <status>1</status>
          <port>N/A</port>
          <pid>28846</pid>
        </node>
        <node>
          <hostname>NFS Server</hostname>
          <path>24b83353-b2d7-4563-b4ea-7ebfacfcf3bb</path>
          <status>1</status>
          <port>38467</port>
          <pid>9487</pid>
        </node>

        ......

        <tasks/>
      </volume>
    </volumes>
  </volStatus>
</cliOutput>


Previously the <path> used to display host name instead of UUID.

--- Additional comment from Kaushal on 2013-04-08 10:46:02 IST ---

Do you still have the glusterd logs for the peer where this command was issued? If you do, can you attach them.

--- Additional comment from Kanagaraj on 2013-04-08 11:02:38 IST ---

Attached the log files from peers.

--- Additional comment from Anand Avati on 2013-04-23 12:18:04 IST ---

REVIEW: http://review.gluster.org/4873 (glusterd: Fix uuid to hostname conversion for 'volume status') posted (#1) for review on master by Kaushal M (kaushal)

--- Additional comment from Anand Avati on 2013-04-23 12:23:53 IST ---

REVIEW: http://review.gluster.org/4874 (glusterd: Fix uuid to hostname conversion for 'volume status') posted (#1) for review on release-3.4 by Kaushal M (kaushal)

--- Additional comment from Anand Avati on 2013-04-23 12:41:01 IST ---

REVIEW: http://review.gluster.org/4873 (glusterd: Fix uuid to hostname conversion for 'volume status') posted (#2) for review on master by Kaushal M (kaushal)

--- Additional comment from Anand Avati on 2013-04-23 12:42:27 IST ---

REVIEW: http://review.gluster.org/4874 (glusterd: Fix uuid to hostname conversion for 'volume status') posted (#2) for review on release-3.4 by Kaushal M (kaushal)

--- Additional comment from Anand Avati on 2013-05-13 12:09:19 IST ---

COMMIT: http://review.gluster.org/4873 committed in master by Vijay Bellur (vbellur) 
------
commit fd36cabb0db4139cba97fc75c6169b57ebea3e9d
Author: Kaushal M <kaushal>
Date:   Tue Apr 23 12:11:00 2013 +0530

    glusterd: Fix uuid to hostname conversion for 'volume status'
    
    Change-Id: I46c41c29c2d11652f6d8ccd5637be0ac9774fc1d
    BUG: 927648
    Signed-off-by: Kaushal M <kaushal>
    Reviewed-on: http://review.gluster.org/4873
    Reviewed-by: Krishnan Parthasarathi <kparthas>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>

--- Additional comment from Anand Avati on 2013-05-14 16:43:59 IST ---

COMMIT: http://review.gluster.org/4874 committed in release-3.4 by Vijay Bellur (vbellur) 
------
commit e6b4c04709a14bff0379ccfc414b8feec35f78fa
Author: Kaushal M <kaushal>
Date:   Tue Apr 23 12:11:00 2013 +0530

    glusterd: Fix uuid to hostname conversion for 'volume status'
    
    BUG: 927648
    Change-Id: Ic016e9d1f090372329a8a2e530dac5fc6ed6c5ae
    Signed-off-by: Kaushal M <kaushal>
    Reviewed-on: http://review.gluster.org/4874
    Reviewed-by: Krishnan Parthasarathi <kparthas>
    Tested-by: Gluster Build System <jenkins.com>
    Reviewed-by: Vijay Bellur <vbellur>