diff options
author | Avra Sengupta <asengupt@redhat.com> | 2016-01-22 16:59:53 +0530 |
---|---|---|
committer | Rajesh Joseph <rjoseph@redhat.com> | 2016-03-11 00:06:55 -0800 |
commit | 7ece0b7b04059f79e25cd7bf44218eb4688dd8e3 (patch) | |
tree | ae5fe613a3027d7de356dab56a381842025d6e89 /geo-replication | |
parent | af7428490253421484e61409ddb77b653c4a5e63 (diff) |
snapshot: Fix parent volinfo corruption in snapshot restore
Backport of http://review.gluster.org/#/c/13282/
Remove parent_volinfo irrespective of success or failure
of glusterd_lvm_snapshot_remove(). This prevents the
duplication of the volinfo after the execution of
glusterd_snapshot_restore().
When commit fails on originator node, we don't perform
commit on the other nodes, as a result we don't have a
backup of /var/lib/glusterd/vols/<volname> in the
GLUSTERD_TRASH in other nodes. But in the postvalidate
we try to restore from GLUSTED_TRASH and end up cleaning
up /var/lib/glusterd/vols/<volname>. Hence moved
glusterd_snapshot_backup_vol() from commit to prevalidate,
so that the backup is always available when a cleanup is
needed.
Change-Id: Icd471b23faf02bad680b9a1aadc4a0175f7cce8b
BUG: 1301030
Signed-off-by: Avra Sengupta <asengupt@redhat.com>
Reviewed-on: http://review.gluster.org/13282
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Rajesh Joseph <rjoseph@redhat.com>
(cherry picked from commit 3f78b6b3dc6376945a1b1a7c4af16103901ab746)
Reviewed-on: http://review.gluster.org/13548
Diffstat (limited to 'geo-replication')
0 files changed, 0 insertions, 0 deletions