Skip to content
Snippets Groups Projects
Commit 01b6558a authored by Iustin Pop's avatar Iustin Pop
Browse files

Clarify the error message for ':' in PV names


As described in issue 93, just saying ':' is not a valid char can be
confusing.

Signed-off-by: default avatarIustin Pop <iustin@google.com>
Reviewed-by: default avatarMichael Hanselmann <hansmi@google.com>
parent 7f69aabb
No related branches found
No related tags found
No related merge requests found
...@@ -389,8 +389,9 @@ class LogicalVolume(BlockDev): ...@@ -389,8 +389,9 @@ class LogicalVolume(BlockDev):
pvlist = [ pv[1] for pv in pvs_info ] pvlist = [ pv[1] for pv in pvs_info ]
if utils.any(pvlist, lambda v: ":" in v): if utils.any(pvlist, lambda v: ":" in v):
_ThrowError("Some of your PVs have invalid character ':'" _ThrowError("Some of your PVs have the invalid character ':' in their"
" in their name") " name, this is not supported - please filter them out"
" in lvm.conf using either 'filter' or 'preferred_names'")
free_size = sum([ pv[0] for pv in pvs_info ]) free_size = sum([ pv[0] for pv in pvs_info ])
current_pvs = len(pvlist) current_pvs = len(pvlist)
stripes = min(current_pvs, constants.LVM_STRIPECOUNT) stripes = min(current_pvs, constants.LVM_STRIPECOUNT)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment