Fix chromium perf fyi tester triggering

Fix chromium perf fyi tester triggering

The parent mastername of fyi builders isn't set for chromium.perf.fyi, which means
the builders which triggers these testers don't know what they're supposed to 
build. And none of this is in expectations anywhere :/

BUG=639530
Committed: https://chromium.googlesource.com/chromium/tools/build/+/7f513989283831ce65f29f73c874e65932168ef0

[martiniss, 2016-10-05 20:06:21]

Description was changed from

==========
Fix chromium perf fyi tester triggering

BUG=639530
==========

to

==========
Fix chromium perf fyi tester triggering

The parent mastername of fyi builders isn't set for chromium.perf.fyi, which
means the builders which triggers these testers don't know what they're supposed
to build. And none of this is in expectations anywhere :/

BUG=639530
==========

[martiniss, 2016-10-05 20:06:34]

Description was changed from

==========
Fix chromium perf fyi tester triggering

The parent mastername of fyi builders isn't set for chromium.perf.fyi, which
means the builders which triggers these testers don't know what they're supposed
to build. And none of this is in expectations anywhere :/

BUG=639530
==========

to

==========
Fix chromium perf fyi tester triggering

The parent mastername of fyi builders isn't set for chromium.perf.fyi, which
means
the builders which triggers these testers don't know what they're supposed to 
build. And none of this is in expectations anywhere :/

BUG=639530
==========

[martiniss, 2016-10-05 20:06:41]

martiniss@chromium.org changed reviewers:
+ dtu@chromium.org, stip@chromium.org

[martiniss, 2016-10-05 20:06:41]

PTAL

[ghost stip (do not use), 2016-10-05 20:55:03]

lgtm

https://codereview.chromium.org/2393263002/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_tests/chromium_perf_fyi.py (right):

https://codereview.chromium.org/2393263002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_tests/chromium_perf_fyi.py:47:
spec['parent_mastername'] = 'chromium.perf.fyi'
apparently only chromium.perf sets parent_buildername

[martiniss, 2016-10-05 20:57:44]

The CQ bit was checked by martiniss@chromium.org

[commit-bot: I haz the power, 2016-10-05 20:57:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-05 21:01:45]

Description was changed from

==========
Fix chromium perf fyi tester triggering

The parent mastername of fyi builders isn't set for chromium.perf.fyi, which
means
the builders which triggers these testers don't know what they're supposed to 
build. And none of this is in expectations anywhere :/

BUG=639530
==========

to

==========
Fix chromium perf fyi tester triggering

The parent mastername of fyi builders isn't set for chromium.perf.fyi, which
means
the builders which triggers these testers don't know what they're supposed to 
build. And none of this is in expectations anywhere :/

BUG=639530

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/7f513989283831ce65f2...
==========

[commit-bot: I haz the power, 2016-10-05 21:01:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/7f513989283831ce65f2...

[dtu, 2016-10-06 00:29:00]

Hm, I thought it defaults to the current master.

[martiniss, 2016-10-06 00:54:28]

On 2016/10/06 at 00:29:00, dtu wrote:
> Hm, I thought it defaults to the current master.

Ah, no. It does.
https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/chromium_...