Re: [PATCH] KVM test: tests.cfg.sample: add params to qemu command line properly

[Avi Kivity <avi@redhat.com>]

  On 09/10/2010 06:57 PM, Lucas Meneghel Rodrigues wrote:
> There's a small bug on the qemu sample testset for KVM autotest:
> We need to append -enable-kvm as extra params to the qemu command
> line, not replace all extra params. Small bug, renders the unattended
> install unusable for folks trying qemu instead of qemu-kvm.
>
>
> @@ -920,11 +920,15 @@ class Job(DBObject):
>                      WHERE afe_job_id = %s
>                      """ % self.id)
>
> -        t_begin, t_end = time_row[0]
> -        delta = t_end - t_begin
> -        minutes, seconds = divmod(delta.seconds, 60)
> -        hours, minutes = divmod(minutes, 60)
> -        stats['execution_time'] = "%02d:%02d:%02d" % (hours, minutes, seconds)
> +        if time_row:
> +            t_begin, t_end = time_row[0]
> +            delta = t_end - t_begin
> +            minutes, seconds = divmod(delta.seconds, 60)
> +            hours, minutes = divmod(minutes, 60)
> +            stats['execution_time'] = "%02d:%02d:%02d" % (hours, minutes,
> +                                                          seconds)
> +        else:
> +            stats['execution_time'] = "00:00:00"

Appears to be unrelated.

-- 
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.