mirror of
https://github.com/gluster/glusterfs.git
synced 2026-02-06 00:49:30 +01:00
glusterfsd/mgmt: Connect backup volfile-server in round robin (#4167)
Once we establish a connection with volfile-server, if that connection looses, then we try to connect to the ony be one till the last one on the list. Once we reach the last one, we never go back to the first one. So if the last one is down we will fail to connect to that node until it comes back Change-Id: I8fe801fb536879d13a3c84eb2d935932b44dbaf5 Fixes: #4166 Signed-off-by: Mohammed Rafi KC <rafi.kavungal@iternity.com>
This commit is contained in:
@@ -819,6 +819,7 @@ mgmt_rpc_notify(struct rpc_clnt *rpc, void *mydata, rpc_clnt_event_t event,
|
||||
rpc_transport_t *rpc_trans = NULL;
|
||||
struct glfs *fs = NULL;
|
||||
int ret = 0;
|
||||
static int log_ctr2;
|
||||
struct dnscache6 *dnscache = NULL;
|
||||
|
||||
this = mydata;
|
||||
@@ -854,13 +855,26 @@ mgmt_rpc_notify(struct rpc_clnt *rpc, void *mydata, rpc_clnt_event_t event,
|
||||
}
|
||||
server = ctx->cmd_args.curr_server;
|
||||
if (server->list.next == &ctx->cmd_args.volfile_servers) {
|
||||
errno = ENOTCONN;
|
||||
gf_smsg("glfs-mgmt", GF_LOG_INFO, ENOTCONN,
|
||||
API_MSG_VOLFILE_SERVER_EXHAUST, NULL);
|
||||
glfs_init_done(fs, -1);
|
||||
break;
|
||||
if (!ctx->active) {
|
||||
errno = ENOTCONN;
|
||||
gf_smsg("glfs-mgmt", GF_LOG_INFO, ENOTCONN,
|
||||
API_MSG_VOLFILE_SERVER_EXHAUST, NULL);
|
||||
glfs_init_done(fs, -1);
|
||||
break;
|
||||
} else {
|
||||
server = list_first_entry(
|
||||
&ctx->cmd_args.volfile_servers, typeof(*server),
|
||||
list);
|
||||
GF_LOG_OCCASIONALLY(log_ctr2, "glusterfsd-mgmt",
|
||||
GF_LOG_INFO,
|
||||
"Exhausted all volfile servers, "
|
||||
"Retrying from again!");
|
||||
}
|
||||
|
||||
} else {
|
||||
server = list_entry(server->list.next, typeof(*server),
|
||||
list);
|
||||
}
|
||||
server = list_entry(server->list.next, typeof(*server), list);
|
||||
ctx->cmd_args.curr_server = server;
|
||||
ctx->cmd_args.volfile_server_port = server->port;
|
||||
ctx->cmd_args.volfile_server = server->volfile_server;
|
||||
|
||||
@@ -2817,13 +2817,21 @@ mgmt_rpc_notify(struct rpc_clnt *rpc, void *mydata, rpc_clnt_event_t event,
|
||||
if (server->list.next == &ctx->cmd_args.volfile_servers) {
|
||||
if (!ctx->active) {
|
||||
need_term = 1;
|
||||
gf_log("glusterfsd-mgmt", GF_LOG_INFO,
|
||||
"Exhausted all volfile servers, Exiting");
|
||||
emval = ENOTCONN;
|
||||
break;
|
||||
} else {
|
||||
server = list_first_entry(&ctx->cmd_args.volfile_servers,
|
||||
typeof(*server), list);
|
||||
emval = ENOTCONN;
|
||||
GF_LOG_OCCASIONALLY(
|
||||
log_ctr2, "glusterfsd-mgmt", GF_LOG_INFO,
|
||||
"Exhausted all volfile servers, Retrying from again!");
|
||||
}
|
||||
emval = ENOTCONN;
|
||||
GF_LOG_OCCASIONALLY(log_ctr2, "glusterfsd-mgmt", GF_LOG_INFO,
|
||||
"Exhausted all volfile servers");
|
||||
break;
|
||||
} else {
|
||||
server = list_entry(server->list.next, typeof(*server), list);
|
||||
}
|
||||
server = list_entry(server->list.next, typeof(*server), list);
|
||||
ctx->cmd_args.curr_server = server;
|
||||
ctx->cmd_args.volfile_server = server->volfile_server;
|
||||
|
||||
|
||||
Reference in New Issue
Block a user