1708665 - Enable reftests on Pixel2 for SW-WR

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, task)

Comment 1

[Jamie Nicol [:jnicol]]

Would it possibly make more sense to run these on the Moto G5? Will need to confirm the device availability on CI, but it might take some pressure off the limited number of Pixel 2s. Plus the Moto g5 run arm32, which is more similar to the devices we expect to need software webrender on.

Comment 2

[Sotaro Ikeda [:sotaro]]

Thanks for the advice!

Comment 3

[Sotaro Ikeda [:sotaro]]

Reftest could run on Pixel2.
https://treeherder.mozilla.org/jobs?repo=try&revision=9c97062139870c9546868d41970a843dcb52d896

But when I tried to run reftest on MotoG5, tests did not run.

https://treeherder.mozilla.org/jobs?repo=try&revision=d908c7285ba110c433268c2324987743a0cb6993
https://treeherder.mozilla.org/jobs?repo=try&revision=f08913f967d569c6e62229ea16d8e69e58a550af

Comment 4

[Sotaro Ikeda [:sotaro]]

:jmaher, :aosmond, :jnicol, do you know what is wrong to run reftests on MotoG5 as in comment 3?

Comment 5

[Sotaro Ikeda [:sotaro]]

(In reply to Sotaro Ikeda [:sotaro] from comment #4)

:jmaher, :aosmond, :jnicol, do you know what is wrong to run reftests on MotoG5 as in comment 3?

Going to enable test with 32bit build on Pixel2 instead of MogoG5.

Comment 6

[Sotaro Ikeda [:sotaro]]

Attachment: Bug 1708665 - Enable reftests on Pixel2 for SW-WR

Comment 7

[Sotaro Ikeda [:sotaro]]

Hmm, screenshot did not work well.
https://treeherder.mozilla.org/jobs?repo=try&revision=9e35d9eb1ee53bf25c87d16fd8c57c81215525bf

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

as for G5 we have very low capacity there- the two try pushes timed out, that is the exception (we have a 24 hour limit to pick up a task). As for P2 we have more capacity for unittests- when we renewed the contract for these phones all jobs were validated and any spare capacity was removed as much as we could. There were no plans to add more jobs.

Comment 9

[Sotaro Ikeda [:sotaro]]

Thank you for the explanation :)

:jmaher, is it OK to add reftests with 32bit debug build to Pixel2?

Comment 10

[Joel Maher ( :jmaher ) (UTC -8)]

if you want it to stick around and not be backed out, I would schedule reftest-qr, not reftest. We just vetted all the jobs running in February, so I doubt there is much we could kick out to make room for duplicating reftest-qr. reftest-qr is intended to be a reduced set of reftests that are most likely to uncover issues on android based on code changes and bugs over time.

Comment 11

[Sotaro Ikeda [:sotaro]]

Attachment: WIP: Bug 1708665 - Enable reftests on Pixel2 for SW-WR

Comment 12

[Sotaro Ikeda [:sotaro]]

reftest result
https://treeherder.mozilla.org/jobs?repo=try&revision=83c51b70c415effce7dc456535caa8c8a0532b70

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/064a300e7408
Enable reftests on Pixel2 for SW-WR r=jmaher

Comment 14

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/064a300e7408