On 21 February 2013 18:51, Michael Hanselmann <[email protected]> wrote:
> Not all hypervisors support the same set of parameters.
> ---
>  qa/qa_instance.py | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/qa/qa_instance.py b/qa/qa_instance.py
> index afdc766..bd160c1 100644
> --- a/qa/qa_instance.py
> +++ b/qa/qa_instance.py
> @@ -435,6 +435,8 @@ def TestInstanceModify(instance):
>      ["-B", "%s=%s" % (constants.BE_ALWAYS_FAILOVER, constants.VALUE_TRUE)],
>      ["-B", "%s=%s" % (constants.BE_ALWAYS_FAILOVER, 
> constants.VALUE_DEFAULT)],
>
> +    # TODO: Select parameters based on hypervisor; not all hypervisors 
> support
> +    # all parameters
>      ["-H", "%s=%s" % (constants.HV_KERNEL_PATH, test_kernel)],
>      ["-H", "%s=%s" % (constants.HV_KERNEL_PATH, constants.VALUE_DEFAULT)],

Wouldn't it make more sense to have a bug open instead?

Bernardo

Reply via email to