diff options
| author | Jonathan Earl Brassow <jbrassow@redhat.com> | 2012-02-22 16:03:55 +0000 |
|---|---|---|
| committer | Jonathan Earl Brassow <jbrassow@redhat.com> | 2012-02-22 16:03:55 +0000 |
| commit | 7a5a5cb06fbdbe338f7481780803aa7c3777c326 (patch) | |
| tree | adbe12354b95ccc9989ca8851928ac87c8160459 /tools/lvconvert.c | |
| parent | 972282bfbc21edde191574ee89eee1f637f0e7d5 (diff) | |
| download | lvm2-7a5a5cb06fbdbe338f7481780803aa7c3777c326.tar.gz lvm2-7a5a5cb06fbdbe338f7481780803aa7c3777c326.tar.xz lvm2-7a5a5cb06fbdbe338f7481780803aa7c3777c326.zip | |
Add some messages that indicate completion of RAID device replacement.
There were no messages printed upon completiion of RAID device replacement.
This could cause confusion/concern during automated recovery, because the
user sees the failure messages but no other messages indicating correction.
Diffstat (limited to 'tools/lvconvert.c')
| -rw-r--r-- | tools/lvconvert.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/tools/lvconvert.c b/tools/lvconvert.c index 5b1ea231..043613f4 100644 --- a/tools/lvconvert.c +++ b/tools/lvconvert.c @@ -1531,7 +1531,15 @@ static int lvconvert_raid(struct logical_volume *lv, struct lvconvert_params *lp if (!(failed_pvs = _failed_pv_list(lv->vg))) return_0; - return lv_raid_replace(lv, failed_pvs, lp->pvh); + if (!lv_raid_replace(lv, failed_pvs, lp->pvh)) { + log_error("Failed to replace faulty devices in" + " %s/%s.", lv->vg->name, lv->name); + return 0; + } + + log_print("Faulty devices in %s/%s successfully" + " replaced.", lv->vg->name, lv->name); + return 1; } /* "warn" if policy not set to replace */ |