Open Bug 2019363 Opened 3 months ago Updated 3 months ago

mach try perf, "Pageload android-a55" includes geckoview example pageload tests

Categories

(Testing :: Performance, task)

Product:
Testing
Component:
Performance
Type:
task

Tracking

(Not tracked)

People

(Reporter: acreskey, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxp])

I ran ./mach try perf selecting Pageload android-a55 and I was surprised to see geckoview example tests in the compare, but not Chrome.

https://hg-edge.mozilla.org/try/rev/3c63bfcbd7644208ff453911291ac6d0e1ec9c59

I don't believe we are interested in geckoview example results in general, so I would consider removing them from ./mach try perf to reduce CI overhhead.

It could be useful to include Chrome in the test list, but I think that's rare, so probably better to make people manually select it.

just making note, last we looked into this (GVE that is), i believe fission folks were interested in GVE performance. Could be worth bringing that discussion up again.

But a quicker solution for this bug would perhaps let Fenix be the default, and put GVE behind a flag like --gve or --geckoview-example so that it is still available for those that are interested.

And yes, Chrome is intentionally not part of the default selection. You have to pass the --chrome flag e.g. ./mach try perf --chrome
https://firefox-source-docs.mozilla.org/testing/perfdocs/mach-try-perf.html#chrome

Whiteboard: [fxp]
You need to log in before you can comment on or make changes to this bug.