* [PATCH] fix question if lvconverting to core log
@ 2009-03-25 13:33 Milan Broz
0 siblings, 0 replies; only message in thread
From: Milan Broz @ 2009-03-25 13:33 UTC (permalink / raw)
To: lvm-devel
yes_no_prompt returns 'n' or 'y'...
Why we require this always? My test case is that PV with log
fails.
Why it need resync before conversion to core log for
inactive mirror?
Signed-off-by: Milan Broz <mbroz@redhat.com>
diff --git a/lib/metadata/mirror.c b/lib/metadata/mirror.c
index 304a8d9..8299c4d 100644
--- a/lib/metadata/mirror.c
+++ b/lib/metadata/mirror.c
@@ -1222,7 +1222,7 @@ int remove_mirror_log(struct cmd_context *cmd,
return 0;
} else if (yes_no_prompt("Full resync required to convert "
"inactive mirror %s to core log. "
- "Proceed? [y/n]: "))
+ "Proceed? [y/n]: ") == 'y')
sync_percent = 0;
else
return 0;
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2009-03-25 13:33 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-25 13:33 [PATCH] fix question if lvconverting to core log Milan Broz
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.