diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2018-01-11 16:15:16 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2018-01-17 14:44:33 +0100 |
commit | 02621cffb0d5ff72adb8770e26c1b4adb95bd0d9 (patch) | |
tree | 56e3d00aa7a24717010de57413f88608cea36e4e | |
parent | 5a961d3411c8c11312d2998378277efc8988fc7f (diff) | |
download | lvm2-02621cffb0d5ff72adb8770e26c1b4adb95bd0d9.tar.gz |
pvmove: drop misleading pvmove restriction for cluster
pvmove handles properly locked LVs in cluster and this extra check
actually cause misbehavior as some LVs were silently skipped from
operation scope.
-rw-r--r-- | tools/pvmove.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/tools/pvmove.c b/tools/pvmove.c index 9bbe12be4..c5d7e52c4 100644 --- a/tools/pvmove.c +++ b/tools/pvmove.c @@ -371,16 +371,6 @@ static struct logical_volume *_set_up_pvmove_lv(struct cmd_context *cmd, if (lv_name && strcmp(lv->name, top_level_lv_name(vg, lv_name))) continue; - /* - * RAID, thin and snapshot-related LVs are not - * processed in a cluster, so we don't have to - * worry about avoiding certain PVs in that context. - * - * Allow clustered mirror, but not raid mirror. - */ - if (vg_is_clustered(vg) && !lv_is_mirror_type(lv)) - continue; - if (!lv_is_on_pvs(lv, source_pvl)) continue; |