LGTM, thanks

On Thu, Sep 24, 2015 at 2:14 PM, 'BSRK Aditya' via ganeti-devel <
[email protected]> wrote:

> The previous default value causes burnin to fail at instance
> creation as the disk size was too low.
>
> Signed-off-by: BSRK Aditya <[email protected]>
> ---
>  lib/tools/burnin.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/tools/burnin.py b/lib/tools/burnin.py
> index 6271701..53ee478 100755
> --- a/lib/tools/burnin.py
> +++ b/lib/tools/burnin.py
> @@ -159,8 +159,8 @@ OPTIONS = [
>    cli.OSPARAMS_OPT,
>    cli.cli_option("--disk-size", dest="disk_size",
>                   help="Disk size (determines disk count)",
> -                 default="128m", type="string", metavar="<size,size,...>",
> -                 completion_suggest=("128M 512M 1G 4G 1G,256M"
> +                 default="1G", type="string", metavar="<size,size,...>",
> +                 completion_suggest=("512M 1G 4G 1G,256M"
>                                       " 4G,1G,1G 10G").split()),
>    cli.cli_option("--disk-growth", dest="disk_growth", help="Disk growth",
>                   default="128m", type="string",
> metavar="<size,size,...>"),
> --
> 2.6.0.rc2.230.g3dd15c0
>
>
Hrvoje Ribicic
Ganeti Engineering
Google Germany GmbH
Dienerstr. 12, 80331, München

Geschäftsführer: Graham Law, Christine Elizabeth Flores
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg

Reply via email to