LGTM, thanks

On Thu, Mar 27, 2014 at 3:22 PM, Klaus Aehlig <[email protected]> wrote:

> Commit 580b1fdd incorrectly assumes that disk parameters are
> just the standard ones, whereas the man page explicitly states
> that additional parameters can be passed as well, if they make
> sense for the chosen storage type. Fix this.
>
> Signed-off-by: Klaus Aehlig <[email protected]>
> Reviewed-by: Petr Pudlak <[email protected]>
>
> Cherry-picked from b26a275a0. Fixes issue 779.
>
> Signed-off-by: Klaus Aehlig <[email protected]>
> ---
>  lib/ht.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/ht.py b/lib/ht.py
> index d6171ac..ee82148 100644
> --- a/lib/ht.py
> +++ b/lib/ht.py
> @@ -570,7 +570,7 @@ TINicParams = \
>                                        TMaybe(TString)))
>
>  TIDiskParams = \
> -    Comment("Disk parameters")(TDictOf(TElemOf(constants.IDISK_PARAMS),
> +    Comment("Disk parameters")(TDictOf(TNonEmptyString,
>                                         TOr(TNonEmptyString, TInt)))
>
>  THypervisor = TElemOf(constants.HYPER_TYPES)
> --
> 1.9.1.423.g4596e3a
>
>

Reply via email to