Fenix performance jobs trigger an unused build of Geckoview (+PGO pass)
Categories
(Testing :: Performance, defect, P3)
Tracking
(Not tracked)
People
(Reporter: acreskey, Unassigned)
References
(Depends on 1 open bug)
Details
(Whiteboard: [ci-costs-2020:todo])
Attachments
(1 obsolete file)
I noticed that when running Fenix performance tests (at least on try), a full PGO build of Geckoview is made.
However this build is not needed since the Fenix performance tests use a baked (nightly) build of Fenix.
To save cloud costs, I believe we can remove the build of Geckoview from Fenix jobs.
These were the tests triggered:
"test-android-hw-g5-7-0-arm7-api-16-shippable/opt-browsertime-tp6m-fenix-cold-bbc-e10s",
"test-android-hw-p2-8-0-arm7-api-16-shippable/opt-browsertime-tp6m-fenix-cold-bbc-e10s"
These Fenix tests also triggered a Geckoview PGO build:
"test-android-hw-g5-7-0-arm7-api-16-shippable/opt-raptor-speedometer-fenix-e10s",
"test-android-hw-p2-8-0-arm7-api-16-shippable/opt-raptor-speedometer-fenix-e10s"
Updated•4 years ago
|
Updated•4 years ago
|
Comment 1•4 years ago
|
||
:bhearsum, is this something you would be able to help us with? I'm not entirely sure what is causing these to get scheduled.
Comment 2•4 years ago
|
||
(In reply to Greg Mierzwinski [:sparky] from comment #1)
:bhearsum, is this something you would be able to help us with? I'm not entirely sure what is causing these to get scheduled.
I'm happy to help! To be clear: it sounds like you're saying that the Fenix performance tests do not need to depend at all on any Android builds? Eg: All of the "Bpgo" jobs in the try push linked don't need to run?
Comment 3•4 years ago
|
||
(In reply to bhearsum@mozilla.com (:bhearsum) from comment #2)
(In reply to Greg Mierzwinski [:sparky] from comment #1)
:bhearsum, is this something you would be able to help us with? I'm not entirely sure what is causing these to get scheduled.
I'm happy to help! To be clear: it sounds like you're saying that the Fenix performance tests do not need to depend at all on any Android builds? Eg: All of the "Bpgo" jobs in the try push linked don't need to run?
Thanks :bhearsum! Yes exactly, we download the APK file from a taskcluster index route instead of building them on the commit:
Comment 4•4 years ago
|
||
Updated•4 years ago
|
Comment 5•4 years ago
|
||
So this is ending up trickier than it looks. Although we don't use the build, we do require the mozharness package from the build to run tests (see https://phabricator.services.mozilla.com/D83369#2564853).
I don't think there's any easy way of moving forward with this at the moment...
Comment 6•4 years ago
|
||
possibly we can be creative and use mozharness.zip from the m-c build we get the test packages from, not sure if that is possible or makes sense.
Comment 7•4 years ago
|
||
Oh that's unfortunate.
Would it be possible to force artifact builds when we have the target field set to a url? (I'm also wondering about what you think about jmaher's idea).
Comment 8•4 years ago
|
||
(In reply to Greg Mierzwinski [:sparky] from comment #7)
Oh that's unfortunate.
Would it be possible to force artifact builds when we have the target field set to a url? (I'm also wondering about what you think about jmaher's idea).
I'll try to dig into these ideas this week or next. Off the top of my head, I'm not sure the viability or difficulty of them.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 10•4 years ago
|
||
:bhearsum, are you still working on this issue?
Updated•4 years ago
|
Comment 11•4 years ago
|
||
(In reply to Greg Mierzwinski [:sparky] from comment #10)
:bhearsum, are you still working on this issue?
Sorry, no. I don't have time to dig into this in the forseeable future.
Updated•4 years ago
|
Updated•4 years ago
|
Description
•