summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlasdair Kergon <agk@redhat.com>2009-05-20 22:24:48 +0000
committerAlasdair Kergon <agk@redhat.com>2009-05-20 22:24:48 +0000
commit450928e30464436235cd91bc59e9c3b118e86ad7 (patch)
tree0e71ecce1e50c192772e9895af545adb4c82579d
parent6ac30c94f2b3aabd679d875f4c490d41549ecb9c (diff)
downloadlvm2-450928e30464436235cd91bc59e9c3b118e86ad7.tar.gz
lvm2-450928e30464436235cd91bc59e9c3b118e86ad7.tar.xz
lvm2-450928e30464436235cd91bc59e9c3b118e86ad7.zip
Revert:
Use lvconvert --repair in dmeventd mirror DSO. for now. It replaces bad behaviour in one set of circumstances with bad behaviour in a different set. We think the behaviour needs to be more configurable.
-rw-r--r--WHATS_NEW1
-rw-r--r--daemons/dmeventd/plugins/mirror/dmeventd_mirror.c2
2 files changed, 1 insertions, 2 deletions
diff --git a/WHATS_NEW b/WHATS_NEW
index 9a98105d..012c2196 100644
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -5,7 +5,6 @@ Version 2.02.46 -
Enable online resizing of mirrors.
Use suspend with flush when device size was changed during table preload.
Introduce CLVMD_CMD_LOCK_QUERY command for clvmd.
- Use lvconvert --repair in dmeventd mirror DSO.
Fix pvmove to revert operation if temporary mirror creation fails.
Fix metadata export for VG with missing PVs.
Add vgimportclone and install it and the man page by default.
diff --git a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c
index 9e89a553..fc610aea 100644
--- a/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c
+++ b/daemons/dmeventd/plugins/mirror/dmeventd_mirror.c
@@ -152,7 +152,7 @@ static int _remove_failed_devices(const char *device)
}
/* FIXME Is any sanity-checking required on %s? */
- if (CMD_SIZE <= snprintf(cmd_str, CMD_SIZE, "lvconvert --config devices{ignore_suspended_devices=1} --repair %s/%s", vg, lv)) {
+ if (CMD_SIZE <= snprintf(cmd_str, CMD_SIZE, "vgreduce --config devices{ignore_suspended_devices=1} --removemissing --force %s", vg)) {
/* this error should be caught above, but doesn't hurt to check again */
syslog(LOG_ERR, "Unable to form LVM command: Device name too long");
dm_pool_empty(_mem_pool); /* FIXME: not safe with multiple threads */