summaryrefslogtreecommitdiffstats
path: root/xlators/mgmt/glusterd/src/glusterd-store.c
diff options
context:
space:
mode:
authorSachin Pandit <spandit@redhat.com>2013-12-24 09:24:42 +0530
committerSachin Pandit <spandit@redhat.com>2014-01-15 08:40:50 +0530
commit9e3459ccf35ac62004e4d8e15e25d1ed264f8904 (patch)
treebf834bd5b8f5f2109a9e2a8fa638ca3c03dbfc9f /xlators/mgmt/glusterd/src/glusterd-store.c
parentc7cab23472b1d59958a0f39bdaf7776e09f039e4 (diff)
mgmt/glusterd : Having a separate list for snapshots.
Creating a separate list for snaps taken, as cluttering snaps in the volume list does not look neat. Change-Id: Ida4a183e95e8694b85ebb5a680d06b7d29a460a0 BUG: 1040947 Signed-off-by: Sachin Pandit <spandit@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-store.c')
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-store.c9
1 files changed, 4 insertions, 5 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-store.c b/xlators/mgmt/glusterd/src/glusterd-store.c
index e26bb1e..16cafdc 100644
--- a/xlators/mgmt/glusterd/src/glusterd-store.c
+++ b/xlators/mgmt/glusterd/src/glusterd-store.c
@@ -1390,9 +1390,9 @@ glusterd_store_perform_snap_volume_store (glusterd_volinfo_t *volinfo,
if (ret)
goto out;
- ret = gf_store_rename_tmppath (snap_volinfo->shandle);
- if (ret)
- goto out;
+ ret = gf_store_rename_tmppath (snap_volinfo->shandle);
+ if (ret)
+ goto out;
out:
if (ret && (fd > 0))
gf_store_unlink_tmppath (volinfo->shandle);
@@ -2992,7 +2992,7 @@ glusterd_store_retrieve_volume (char *volname, glusterd_snap_t *snap)
// as of now snap volume are also added to the list of volume
volinfo->is_snap_volume = _gf_true;
snap->snap_volume = volinfo;
- list_add_tail (&volinfo->vol_list, &priv->volumes);
+ list_add_tail (&volinfo->vol_list, &priv->snap_list);
}
@@ -3222,7 +3222,6 @@ glusterd_store_retrieve_snap_list (char *volname)
gf_store_handle_t *shandle = NULL;
priv = THIS->private;
-
ret = glusterd_volinfo_find (volname, &volinfo);
if (ret) {
gf_log (THIS->name, GF_LOG_ERROR, "Couldn't get"