[Bugs] [Bug 1379544] New: glusterd creates all rpc_clnts with the same name

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1379544

            Bug ID: 1379544
           Summary: glusterd creates all rpc_clnts with the same name
           Product: GlusterFS
           Version: mainline
         Component: glusterd
          Keywords: Triaged
          Severity: low
          Priority: low
          Assignee: bugs at gluster.org
          Reporter: kaushal at redhat.com
                CC: bugs at gluster.org, kaushal at redhat.com



Description of problem:
GlusterD create rpc_clnts for every peer it connects to. But all the rpc_clnts
are created with `this->name`, ie. GlusterD's xlator name '0-management', as
the name.

This makes it problematic when attempting to identify socket_connect failures
of GlusterD rpc_clnts. `socket_connect()` logs connect failures with,
```
gf_log (this->name, ((sa_family == AF_UNIX) ? GF_LOG_DEBUG : GF_LOG_ERROR),
"connection attempt on %s failed, (%s)", this->peerinfo.identifier, strerror
(errno));
```

This log is basically useless for identifying connect to which remote endpoint
failed, as `this->name` is the same for all GlusterD rpc_clnt transports and
`peerinfo->identifier` is only filled after a successful connect.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.