LGTM, thanks!

On Mon, Oct 6, 2014 at 8:11 PM, Yuto KAWAMURA(kawamuray) <
[email protected]> wrote:

> Currently there are no hvparam modification tests for the LXC
> hypervisor, so we should add additional LXC-specific parameters that can
> be used for instance modification tests of hypervisor parameters.
>
> Signed-off-by: Yuto KAWAMURA(kawamuray) <[email protected]>
> ---
>  qa/qa_instance.py | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/qa/qa_instance.py b/qa/qa_instance.py
> index 23aba21..cecec18 100644
> --- a/qa/qa_instance.py
> +++ b/qa/qa_instance.py
> @@ -643,6 +643,13 @@ def TestInstanceModify(instance):
>        ["--disk", "-1:add,size=1G", "--hotplug"],
>        ["--disk", "-1:remove", "--hotplug"],
>        ])
> +  elif default_hv == constants.HT_LXC:
> +    args.extend([
> +      ["-H", "%s=0" % constants.HV_CPU_MASK],
> +      ["-H", "%s=%s" % (constants.HV_CPU_MASK, constants.VALUE_DEFAULT)],
> +      ["-H", "%s=0" % constants.HV_LXC_TTY],
> +      ["-H", "%s=%s" % (constants.HV_LXC_TTY, constants.VALUE_DEFAULT)],
> +      ])
>
>    url = "http://example.com/busybox.img";
>    args.extend([
> --
> 2.0.4
>
>

Reply via email to