Bug 1548459 - Use suite name to initialize brower cycle combinations. r=perftest-reviewers,rwood

This patch fixes an issue where if browser_cycle does not equal 1 for the first entry in the raptor results, then it will attempt to access combined_suites with a key entry which would only get created when we had an entry with browser_cycle equal to 1. With this change, we use the suite name to determine when we need to initialize the combined_suites entry. This issue occurs when the first cycle does not produce results.

Differential Revision: https://phabricator.services.mozilla.com/D43060

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Gregory Mierzwinski 2019-08-22 18:14:42 +00:00
Родитель 23d547abce
Коммит f528283d3d
1 изменённых файлов: 1 добавлений и 1 удалений

Просмотреть файл

@ -245,7 +245,7 @@ class Output(object):
browser_cycle = next_suite['details']['browser_cycle']
LOG.info("combining results from browser cycle %d for %s"
% (browser_cycle, suite_name))
if browser_cycle == 1:
if suite_name not in combined_suites:
# first browser cycle so just take entire entry to start with
combined_suites[suite_name] = next_suite['details']
LOG.info("created new combined result with intial cycle replicates")