812609 - cedar mozharness talos not running win7 xperf

Status: RESOLVED FIXED

(Release Engineering :: Applications: MozharnessCore, defect, P3)

Description

[Aki Sasaki (not active)]

We currently run a win7-specific talos suite, xperf.
I don't currently run this on Cedar, nor do I have support in either buildbot-configs or talos.json for a platform-specific (or limited platform) suite.

We should add this before we roll out.

Comment 2

[Aki Sasaki (not active)]

This is 'x' on tbpl.

Comment 3

[Jeff Hammel]

What is the plan to proceed on this (or is there one yet)?

Comment 4

[Aki Sasaki (not active)]

(In reply to Aki Sasaki [:aki] from comment #0)
> We currently run a win7-specific talos suite, xperf.
> I don't currently run this on Cedar, nor do I have support in either
> buildbot-configs or talos.json for a platform-specific (or limited platform)
> suite.
> 
> We should add this before we roll out.

Todo: add platform-limiting support to buildbot-configs and talos.json.

Comment 5

[Armen [:armenzg]]

Attachment: enable xperf on ash

Comment 6

[Aki Sasaki (not active)]

Comment on attachment 756697 [details] [diff] [review]
enable xperf on ash

Hm, does this work?

I forget how I did things, but it's looking like I didn't touch the talos suites in buildbot-configs and did things through buildbotcustom and talos.json.

It's possible you'll have to add xperf config info into ash's talos.json.

Comment 7

[Armen [:armenzg]]

(In reply to Aki Sasaki [:aki] from comment #6)
> Comment on attachment 756697 [details] [diff] [review]
> enable xperf on ash
> 
> Hm, does this work?
> 
> I forget how I did things, but it's looking like I didn't touch the talos
> suites in buildbot-configs and did things through buildbotcustom and
> talos.json.
> 
> It's possible you'll have to add xperf config info into ash's talos.json.

It adds the builder which was missing.
The entry is already on the talos.json.

Comment 8

[Rail Aliiev [:rail]]

in production

Comment 9

[Jason Yeo[:jyeo]]

Attachment: enable xperf on cedar

Comment 10

[Armen [:armenzg]]

Comment on attachment 772657 [details] [diff] [review]
enable xperf on cedar

Review of attachment 772657 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.
Feel free to go ahead and land it.

Comment 11

[Justin Wood (:Callek)]

Comment on attachment 756697 [details] [diff] [review]
enable xperf on ash

In production

Comment 12

[Justin Wood (:Callek)]

err actually it looks like the in production one is cedar

Comment 13

[Jason Yeo[:jyeo]]

Attachment: s/tp5o/tp5n for xperf

We should be using the same values that ash's talos.json is using. http://hg.mozilla.org/projects/ash/file/dcdcafb5dd2e/testing/talos/talos.json

Comment 14

[Jason Yeo[:jyeo]]

Attachment: s/tp5o/tp5n for xperf

Reposting. (I forgot to check the patch checkbox)

Comment 15

[Jason Yeo[:jyeo]]

Awesome! xperf jobs are running and looking green on cedar. Fixed.