On Mon, 2011-05-23 at 09:47 -0400, Cleber Rosa wrote:
> This change avoids looking at other irrelevant tupple entries
> and limits the finding of failed tests to the correct tupple
> entry.
> 
> Signed-off-by: Cleber Rosa <[email protected]>
> ---
>  scheduler/scheduler_models.py |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/scheduler/scheduler_models.py b/scheduler/scheduler_models.py
> index 870914d..3675397 100644
> --- a/scheduler/scheduler_models.py
> +++ b/scheduler/scheduler_models.py
> @@ -895,7 +895,7 @@ class Job(DBObject):
>                  ORDER BY t.reason
>                  """ % self.id)
>  
> -        failed_rows = [r for r in rows if not 'GOOD' in r]
> +        failed_rows = [r for r in rows if not r[1] == 'GOOD']

^ Ok, good one, applied!

>          n_test_jobs = _find_test_jobs(rows)
>          n_test_jobs_failed = _find_test_jobs(failed_rows)


_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to