[pykickstart][PATCH] Add support for specifying thin pool profile
Chris Lumens
clumens at redhat.com
Mon Sep 22 15:10:21 UTC 2014
> > Is it possible we will one day want to add something else to the logvol
> > command that could be named "profile"? I feel the current option name
> > is too generic and we'll regret it later.
> Not much possible, as the '--profile' option is a generic option for the
> 'lvm' command documented in lvm (8). Thus I think it won't be ever used
> for anything else.
Definitely - profile is something used by LVM. However the man page
doesn't say it is specific to thinp support. The pykickstart patch here
makes it an error to use --profile without --thinpool. What I am
wondering is if that's too strict. If the intention is to have a
--profile that must be used with --thinpool, perhaps the option should
be named differently. On the other hand, if we will later use --profile
for some other LVM-related purpose, perhaps the check for --thinpool
also being specified does not need to be there.
- Chris
More information about the anaconda-patches
mailing list