New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 833468 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----

Blocking:
issue 829949



Sign in to add a comment

In flake_try_job.ScheduleFlakeTryJob, parameters.dimensions should not be None

Project Member Reported by st...@chromium.org, Apr 16 2018

Issue description

I don't understand, it seems it is not None, and it still fails.
notnone.png
222 KB View Download

Comment 4 by st...@chromium.org, Apr 24 2018

I don't understand either.

But the call stack in the error showed that parameters.dimensions is None.

AttributeError: 'NoneType' object has no attribute 'ToSerializable'
at ScheduleFlakeTryJob (/base/data/home/apps/s~findit-for-me/waterfall-backend:14943-ca594cf.408981509714524873/services/flake_failure/flake_try_job.py:312)
at RunImpl (/base/data/home/apps/s~findit-for-me/waterfall-backend:14943-ca594cf.408981509714524873/pipelines/flake_failure/run_flake_try_job_pipeline.py:81)
at run (/base/data/home/apps/s~findit-for-me/waterfall-backend:14943-ca594cf.408981509714524873/gae_libs/pipelines.py:420)
at _run_internal (/base/data/home/apps/s~findit-for-me/waterfall-backend:14943-ca594cf.408981509714524873/infra_api_clients/../third_party/pipeline/pipeline.py:1113)
at evaluate (/base/data/home/apps/s~findit-for-me/waterfall-backend:14943-ca594cf.408981509714524873/infra_api_clients/../third_party/pipeline/pipeline.py:2158)
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 25 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/8dfb290cb675e6e4dd53c12f484e144939e9c893

commit 8dfb290cb675e6e4dd53c12f484e144939e9c893
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Wed Apr 25 20:42:38 2018

[Findit] Flake Analyzer - Fixing nonetype for try job dimensions

waterfall_config may return None as the try bot dimensions. When bubbling
this up to the caller code, ListOfBasestring.FromSerializable(None) will also
be none. Thus when passing this to services/try_job.py.TriggerTryJob, which
expects a list, None is called with ToSerializable() causing the exception.

Bug:  833468 
Change-Id: Ia13c36a45f2cdcd7181d5714ce9d139194d40e5b
Reviewed-on: https://chromium-review.googlesource.com/1026457
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>
Reviewed-by: Shuotao Gao <stgao@chromium.org>

[modify] https://crrev.com/8dfb290cb675e6e4dd53c12f484e144939e9c893/appengine/findit/services/flake_failure/flake_try_job.py

Status: Fixed (was: Available)

Sign in to add a comment