1597711 - Add conditioned profile support for GeckoView in Raptor

Status: RESOLVED FIXED

(Testing :: Raptor, task, P1)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Need to add conditioned-profile support for mobile browsers (starting with GeckoView), in Raptor.

xref: bug 1547747

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

I've started working on this, but have quite a bit of platform and mobile-app detecting, etc., to go, still (hardcoded hacks ahead): https://treeherder.mozilla.org/#/jobs?repo=try&selectedJob=278154723&revision=4fd1eb04abf0fa2163b5fa88e111e85eae9ed708

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Bug 1597711 - Add conditioned profile support for GV & Fenix in Raptor

Comment 3

[Tarek Ziadé (:tarek)]

this is the code you need:

if isinstance(self, PerftestAndroid):
  platform = "%s-%s" % (self.config["device_name"], self.config["binary"].split("org.mozilla.")[-1])
else:
  platform = get_current_platform()

with a new argument --device-name to add into the raptor CLI (like how you did with --with-conditioned-profile)
then you can add --device-name=XXX mimicing what is done here : https://searchfox.org/mozilla-central/source/taskcluster/ci/condprof/kind.yml#171

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: device_name.patch

The attached patch is as far as I've gotten for today; it's linty fresh, but unittests which download conditioned profiles fail due to device_name still being None.

To test, I've been running both ./mach raptor-test -t raptor-tp6m-1-cold --device-name --app geckoview --binary org.mozilla.geckoview_example for mobile and ./mach raptor-test -t raptor-tp6-1-cold for desktop.

Desktop runs and passes fine, but I haven't tested extensively nor pushed yet to try. I've gotten --device-name being read in from the commandline, as a test, but am hitting 07:56:53 INFO - raptor.py: error: argument --device-name: expected one argument with the patch I've just attached, here. (Didn't want to spam Phab with it yet.)

Comment 5

[Alexandru Ionescu (needinfo me) [:alexandrui]]

I imported the WIP patch and then I tried to get the second attached, but I am receiving some conflicts. Not sure if there are any other patches/changes in order to make this work. Will continue trying.

Edit: still didn't manage. If you to a rebase to central and send me a hg diff -r [latest rev on central] -r [tip of your bookmark] yould really help me.

Comment 7

[Tarek Ziadé (:tarek)]

try run https://treeherder.mozilla.org/#/jobs?repo=try&tier=1%2C2%2C3&revision=3fe59fb502398422f36688345a7883cd53b32799

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to Alexandru Ionescu :alexandrui from comment #5)

I imported the WIP patch and then I tried to get the second attached, but I am receiving some conflicts. Not sure if there are any other patches/changes in order to make this work. Will continue trying.

Edit: still didn't manage. If you to a rebase to central and send me a hg diff -r [latest rev on central] -r [tip of your bookmark] yould really help me.

Hey, sorry about this; Tarek's handling this patch, now, and has it well in hand.

Comment 9

[Tarek Ziadé (:tarek)]

try run https://treeherder.mozilla.org/#/jobs?repo=try&tier=1%2C2%2C3&revision=06b35390dc453f9b2bc1550c37dde32dd9dae84e

Comment 10

[Tarek Ziadé (:tarek)]

new try https://treeherder.mozilla.org/#/jobs?repo=try&tier=1%2C2%2C3&revision=4cac49f5ea6f27d4e4e083c7407fae208d9b8cd4

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cb86af33e59f
Add conditioned profile support for GV & Fenix in Raptor r=stephendonner,rwood

Comment 12

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/cb86af33e59f