diff options
author | Pranith Kumar K <pkarampu@redhat.com> | 2019-10-24 12:24:35 +0530 |
---|---|---|
committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2019-10-30 05:56:22 +0000 |
commit | b6da637929be0bdcdb00bf38fa4c14a8a09209d3 (patch) | |
tree | 0d8740f13f44be9378654dd66518871d298a801c /libglusterfs/src/events.c | |
parent | f221ba83e11fced96b5a7b506e692358ed278ca2 (diff) |
gf-event: Handle unix volfile-servers
Problem:
glfsheal program uses unix-socket-based volfile server.
volfile server will be the path to socket in this case.
gf_event expects this to be hostname in all cases. So getaddrinfo
will fail on the unix-socket path, events won't be sent in this case.
Fix:
In case of unix sockets, default to localhost
fixes: bz#1765017
Change-Id: I60d27608792c29d83fb82beb5fde5ef4754bece8
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Diffstat (limited to 'libglusterfs/src/events.c')
-rw-r--r-- | libglusterfs/src/events.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/libglusterfs/src/events.c b/libglusterfs/src/events.c index 9d337831387..4e2f8f93a00 100644 --- a/libglusterfs/src/events.c +++ b/libglusterfs/src/events.c @@ -43,6 +43,7 @@ _gf_event(eventtypes_t event, const char *fmt, ...) struct addrinfo *result = NULL; xlator_t *this = THIS; int sin_family = AF_INET; + char *volfile_server_transport = NULL; /* Global context */ ctx = THIS->ctx; @@ -62,8 +63,16 @@ _gf_event(eventtypes_t event, const char *fmt, ...) memset(&hints, 0, sizeof(hints)); hints.ai_family = AF_UNSPEC; + if (ctx) { + volfile_server_transport = ctx->cmd_args.volfile_server_transport; + } + + if (!volfile_server_transport) { + volfile_server_transport = "tcp"; + } /* Get Host name to send message */ - if (ctx && ctx->cmd_args.volfile_server) { + if (ctx && ctx->cmd_args.volfile_server && + (strcmp(volfile_server_transport, "unix"))) { /* If it is client code then volfile_server is set use that information to push the events. */ if ((getaddrinfo(ctx->cmd_args.volfile_server, NULL, &hints, |