From 71191eae5c90e7a5397b6d3ac4bf6bdab36180af Mon Sep 17 00:00:00 2001 From: Alasdair Kergon Date: Wed, 20 May 2009 22:24:48 +0000 Subject: [PATCH] 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. --- WHATS_NEW | 1 - daemons/dmeventd/plugins/mirror/dmeventd_mirror.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/WHATS_NEW b/WHATS_NEW index 9a98105d2..012c21968 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 9e89a553e..fc610aea8 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 */ -- 2.43.5