diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2014-09-04 13:31:17 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2014-09-04 13:31:17 -0500 |
commit | c3d7a14ad4b1d0dfa5944224390d55b9effd00ed (patch) | |
tree | ca977147879514d3ffa11083309de4c244b2e920 | |
parent | 5b63736e53b7a8d4222a86699accecaf2ef856e6 (diff) | |
download | lvm2-c3d7a14ad4b1d0dfa5944224390d55b9effd00ed.tar.gz |
toollib: print ignoring vorigin
When ignoring 'listed' volume, print info message.
(So the final command error message is a bit less confusing,
i.e. when user tries to deactive virtual origin:
> lvchange -an vg/lvol2_vorigin
Ignoring virtual origin logical volume vg/lvol2_vorigin.
One or more specified logical volume(s) not found.
(Reapplied after revert.)
-rw-r--r-- | tools/toollib.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/tools/toollib.c b/tools/toollib.c index 110a06286..b65a8100d 100644 --- a/tools/toollib.c +++ b/tools/toollib.c @@ -1492,8 +1492,14 @@ int process_each_lv_in_vg(struct cmd_context *cmd, lv_is_cow(lvl->lv) && !lv_is_virtual_origin(origin_from_cow(lvl->lv))) continue; - if (lv_is_virtual_origin(lvl->lv) && !arg_count(cmd, all_ARG)) + if (lv_is_virtual_origin(lvl->lv) && !arg_count(cmd, all_ARG)) { + if (lvargs_supplied && + str_list_match_item(arg_lvnames, lvl->lv->name)) + log_print_unless_silent("Ignoring virtual origin logical + display_lvname(lvl->lv)); + continue; + } /* * Only let hidden LVs through it --all was used or the LVs |