Closed Bug 1495798 Opened 2 years ago Closed 2 years ago

sccache gets shutdown after the build, before `make check` in rusttest builds

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox-esr60 fixed, firefox64 fixed)

RESOLVED FIXED
mozilla64
Tracking Status
firefox-esr60 --- fixed
firefox64 --- fixed

People

(Reporter: ted, Assigned: ted)

References

Details

Attachments

(1 file)

I looked at one of the logs from bug 1425365 but unfortunately the sccache log didn't contain useful info. As it turns out, that's because we shut down sccache after the build:
https://dxr.mozilla.org/mozilla-central/rev/56b988a937689d5599400afa59b72c390b40abf2/client.mk#160

However, in rusttest builds when we run `make check` we wind up building more Rust code which will start sccache again, but without logging enabled, which is unfortunate.

I guess the best fix here would be to move the sccache shutdown into mozharness so it runs after the build and check steps.
Blocks: 1396845
Assignee: nobody → ted
In builds that use sccache, the sccache server would be shut down at the end
of the build step in client.mk. Unfortunately in rusttest builds the check
step winds up compiling more Rust code which restarts the sccache server but
without the proper configuration. This patch moves sccache shutdown to a
post-script step in mozharness so that the server will stay running through
the check step.
This is looking pretty good on try so far. I'm retriggering lots of win64-rusttest jobs to see if any of the bugs in the dependency graph crop up.
No longer blocks: 1396845
Pushed by tmielczarek@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/47bcc150ba0b
shut down sccache server in mozharness after all build steps. r=firefox-build-system-reviewers,gps
https://hg.mozilla.org/mozilla-central/rev/47bcc150ba0b
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.