diff options
| author | Atin Mukherjee <amukherj@redhat.com> | 2017-12-06 18:05:24 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2017-12-07 04:49:26 +0000 |
| commit | 4390dededf3d5befceba5e28202d8c48e8608fad (patch) | |
| tree | 179bce75a3c95705a05ccc491e32ac056148c308 | |
| parent | aee30521e006a21b8f8d9a65ffe40d87125ccc17 (diff) | |
glusterd: Free up svc->conn on volume delete
Daemons like snapd, tierd and gfproxyd are maintained on per volume
basis and on a volume delete we should destroy the rpc connection
established for them.
>mainline patch : https://review.gluster.org/#/c/18957/
Change-Id: Id1440e39da07b990fdb9b207df18da04b1ca8014
BUG: 1523046
Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
(cherry picked from commit 36ce4c614a3391043a3417aa061d0aa16e60b2d3)
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-utils.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c index 0ca705c11db..43109bdcea6 100644 --- a/xlators/mgmt/glusterd/src/glusterd-utils.c +++ b/xlators/mgmt/glusterd/src/glusterd-utils.c @@ -1028,6 +1028,11 @@ glusterd_volinfo_delete (glusterd_volinfo_t *volinfo) if (volinfo->rebal.dict) dict_unref (volinfo->rebal.dict); + /* Destroy the connection object for per volume svc daemons */ + glusterd_conn_term (&volinfo->snapd.svc.conn); + glusterd_conn_term (&volinfo->tierd.svc.conn); + glusterd_conn_term (&volinfo->gfproxyd.svc.conn); + gf_store_handle_destroy (volinfo->quota_conf_shandle); gf_store_handle_destroy (volinfo->shandle); gf_store_handle_destroy (volinfo->node_state_shandle); |