diff options
| author | Sachin Pandit <spandit@redhat.com> | 2013-12-24 09:24:42 +0530 |
|---|---|---|
| committer | Sachin Pandit <spandit@redhat.com> | 2014-01-15 08:40:50 +0530 |
| commit | 9e3459ccf35ac62004e4d8e15e25d1ed264f8904 (patch) | |
| tree | bf834bd5b8f5f2109a9e2a8fa638ca3c03dbfc9f /xlators/mgmt/glusterd/src/glusterd-store.c | |
| parent | c7cab23472b1d59958a0f39bdaf7776e09f039e4 (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.c | 9 |
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 e26bb1e22..16cafdc8e 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" |