summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZdenek Kabelac <zkabelac@redhat.com>2016-04-25 13:40:26 +0200
committerZdenek Kabelac <zkabelac@redhat.com>2016-04-26 23:28:01 +0200
commitd8bdc7af3749cbdc9110057daa837d2fe19dcf72 (patch)
tree60740a9def40e817e16bd0b52927725bbaba354a
parent84a5b19539483e5f06b4a1405bb76e46ce9226b2 (diff)
downloadlvm2-d8bdc7af3749cbdc9110057daa837d2fe19dcf72.tar.gz
lvchange: fix return code
We already opened VG for this - so not an invalid cmdline.
-rw-r--r--WHATS_NEW1
-rw-r--r--tools/lvchange.c2
2 files changed, 2 insertions, 1 deletions
diff --git a/WHATS_NEW b/WHATS_NEW
index b6d9fd2ff..d55745994 100644
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
Version 2.02.152 -
==================================
+ Use failed command return code when lvchanging read-only volume.
Show creation transaction_id and zeroing state of pool with thin volume.
Stop checking for dm_cache_mq policy with cache target 1.9 (alias to smq).
Check first /sys/module/dm_* dir existance before using modprobe.
diff --git a/tools/lvchange.c b/tools/lvchange.c
index 631d7f860..0f46e2c8f 100644
--- a/tools/lvchange.c
+++ b/tools/lvchange.c
@@ -915,7 +915,7 @@ static int _lvchange_single(struct cmd_context *cmd, struct logical_volume *lv,
arg_count(cmd, metadataprofile_ARG))) {
log_error("Only -a permitted with read-only volume "
"group \"%s\"", lv->vg->name);
- return EINVALID_CMD_LINE;
+ return ECMD_FAILED;
}
if (lv_is_origin(lv) && !lv_is_thin_volume(lv) &&