New issue
Advanced search Search tips

Issue 596934 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 596145
Owner: ----
Closed: Mar 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

issue 457078

Sign in to add a comment

sizes step failing on win_perf_bisect_builder

Project Member Reported by, Mar 22 2016

Issue description

For example:

  File "C:\b\build\scripts\slave\", line 1951, in <module>
  File "C:\b\build\scripts\slave\", line 1916, in main
    result = _MainWin(options, args, extra_env)
  File "C:\b\build\scripts\slave\", line 1489, in _MainWin
  File "C:\b\build\scripts\slave\", line 123, in annotate
    raise Exception(' error: perf step specified but'
Exception: error: perf step specified butno test_id in factory_properties!

Quinten, you added this exception. How do I make it go away?

I don't need this data to be uploaded to the perf dashboard, so the step is running without results_url or perf_id (see

Comment 1 by, Mar 22 2016

Mergedinto: 596145
Status: Duplicate (was: Available)
Hmm, the error message is suggesting that if the "log processor" object has the method PerformanceLogs, then it's assumed to be a "perf step".

A few possible changes:

 - Remove the check and the exception, and instead just log a warning and don't write performance log lines
 - Find a different way to check whether we want perf results besides looking at the log processor object
 - Find a way to make it use a different log processor object
 hasattr(log_processor, 'PerformanceLogs')

Do we still want to make one of these changes?

(Note, I didn't actually add that exception - it has been there for a long time (more than 2 and a half years); it was in before it was moved to

Sign in to add a comment