Leave PGO decision to mozconfigs and taskcluster, not mozharness
Categories
(Firefox Build System :: General, enhancement)
Tracking
(firefox67 fixed, firefox68 fixed)
People
(Reporter: glandium, Assigned: glandium)
References
(Depends on 1 open bug)
Details
Attachments
(4 files)
See discussion in bug 1534272.
Comment 1•6 years ago
|
||
Taskcluster has authorative information about the repository being built, so
pass that to mozharness, rather than have mozharness reconstruct it from
hand-maintained mozharness config.
Comment 2•6 years ago
|
||
The only thing this affects is the default update channel, but for shipping builds, this is set
explicitly via taskcluster, and for other branches, defaults to default
.
Comment 3•6 years ago
|
||
The release mozconfigs set MOZ_PGO
and all shipping builds set the nightly
mozharness config, so we don't need to additionaly set PGO based on the branch.
Comment 4•6 years ago
|
||
We have moved per-branch configuration to taskcluster, so we can remove that
support in mozharness.
Comment 6•6 years ago
|
||
(In reply to Andreea Pavel [:apavel] from comment #5)
Can this be landed?
Not yet. I still need to do some testing on this.
Assignee | ||
Comment 7•6 years ago
|
||
If you're asking for bug 1534272, I think we should go with the patch in that bug for beta.
Comment 8•6 years ago
|
||
Yes, that's why I asked. Will wait until you guys give the go ahead.
Thank you.
Comment 10•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/775f40c959fc
https://hg.mozilla.org/mozilla-central/rev/db11228c8a38
https://hg.mozilla.org/mozilla-central/rev/3e4b58569126
https://hg.mozilla.org/mozilla-central/rev/d5c2b9a1c215
Comment 11•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/bb986fa5395d
https://hg.mozilla.org/releases/mozilla-beta/rev/3c0a45288132
https://hg.mozilla.org/releases/mozilla-beta/rev/5903d728d99a
https://hg.mozilla.org/releases/mozilla-beta/rev/3a81fd4a729b
Comment 13•6 years ago
|
||
Description
•