Closed
Bug 1384759
Opened 7 years ago
Closed 7 years ago
Perform builds for Servo checkins
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(firefox56 fixed)
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: gps, Assigned: gps)
References
Details
Attachments
(1 file)
We've long had code in taskgraph that only runs tasks for an allow list of platforms for Servo vendoring changes. Now that we build Servo code into Firefox, this filtering is too aggressive and we're failing to find build bustage on some platforms when Servo code is updated.
Comment 1•7 years ago
|
||
Note that the majority of directories in servo/ aren't built. The Right Thing to do would be to filter based on the Cargo.lock file in toolkit/library/rust, which is the canonical indicator of the crates we build. We could also do a dumber manual list of stuff we definitely won't be pulling in anytime soon, or we could just always build.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•7 years ago
|
||
The Decision task is just a bunch of Python code. It doesn't (yet) do anything related to the build system. If we were to intelligently filter changes through Cargo.lock, what would that look like? I assume it involves some subset of "reinvent Cargo?" We probably want to error on the side of caution. i.e. run jobs if there's any doubt. If doing this check correctly is too difficult, it might be best to remove the filter and take the capacity hit :/
Comment 4•7 years ago
|
||
The file looks like this: http://searchfox.org/mozilla-central/source/toolkit/library/rust/Cargo.lock I think we could do something like: If the changes are in servo/components/foo and there is not a line of the form |name = "foo"| in Cargo.lock, they are NPOTB. Worth running that by somebody else, but it seems like it would work.
![]() |
||
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8890601 [details] Bug 1384759 - Only filter test tasks for Servo changes; https://reviewboard.mozilla.org/r/161762/#review167420 I could have sworn that there are tests related to this filter, and tweaking the filter would thereby break those tests, but I cannot find them right now. I think bholley is right that we could be smarter, but we can at least land this change to start catching build bustage. We can then make things smarter if we find we're building too much (who wants to write the Cargo.lock analysis for the last three months or so?). My impression is that the commit volume for Servo is both low enough to not be worried about it, and the majority of commits right now touch code that we would care about anyway, so smarter filtering wouldn't buy that much.
Attachment #8890601 -
Flags: review?(nfroyd) → review+
Pushed by gszorc@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/13358d0eb5a1 Only filter test tasks for Servo changes; r=froydnj
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/13358d0eb5a1
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•