[lvm-devel] [PATCH] Print less scary message if pvmove already finished.
Milan Broz
mbroz at redhat.com
Wed Jan 19 22:37:12 UTC 2011
If other process finishes (ar aborts) pvmove operation and
polling function cannot find any lv with PVMOVE flag return
success and do not print "aborting" message.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=602389
---
tools/polldaemon.c | 12 ++++++++++--
1 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/tools/polldaemon.c b/tools/polldaemon.c
index aac9c17..fdb3edd 100644
--- a/tools/polldaemon.c
+++ b/tools/polldaemon.c
@@ -193,8 +193,16 @@ static int _wait_for_single_lv(struct cmd_context *cmd, const char *name, const
return 0;
}
- if (!(lv = parms->poll_fns->get_copy_lv(cmd, vg, name, uuid,
- parms->lv_type))) {
+ lv = parms->poll_fns->get_copy_lv(cmd, vg, name, uuid, parms->lv_type);
+
+ if (!lv && parms->lv_type == PVMOVE) {
+ log_print("%s: pvmove finished or aborted by other process.",
+ name);
+ unlock_and_free_vg(cmd, vg, vg->name);
+ return 1;
+ }
+
+ if (!lv) {
log_error("ABORTING: Can't find LV in %s for %s",
vg->name, name);
unlock_and_free_vg(cmd, vg, vg->name);
--
1.7.2.3
More information about the lvm-devel
mailing list