Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set SCCACHE_IDLE_TIMEOUT=0 when starting the sccache server #3403

Closed
wants to merge 1 commit into from

Conversation

@luser
Copy link
Contributor

luser commented Dec 12, 2018

This will prevent the sccache server from shutting down due to a lack of activity which can manifest as build failures. This is effectively a workaround for this sccache issue: mozilla/sccache#204 .


This change is Reviewable

This will prevent the sccache server from shutting down due to a lack of activity which can manifest as build failures. This is effectively a workaround for this sccache issue: mozilla/sccache#204 .
@staktrace
Copy link
Contributor

staktrace commented Dec 12, 2018

Thanks, this is good to know, but we're also still hitting #3248 (comment) so at the moment I'd still like to leave sccache disabled until it's more reliable.

(Also note that the .taskcluster.yml file has two different jobs which use sccache so both of them would need this fix applied.)

@bors-servo
Copy link
Contributor

bors-servo commented Dec 12, 2018

The latest upstream changes (presumably #3402) made this pull request unmergeable. Please resolve the merge conflicts.

@gw3583
Copy link
Collaborator

gw3583 commented Dec 18, 2018

@staktrace @luser Do we want to merge this (after a rebase) or should this be closed?

@staktrace
Copy link
Contributor

staktrace commented Dec 18, 2018

Unless we know how to resolve #3248 (comment) I don't think it's worth re-enabling sccache. So I'd lean towards closing this for now.

@gw3583
Copy link
Collaborator

gw3583 commented Dec 18, 2018

OK, feel free to reopen if we want to reconsider landing this.

@gw3583 gw3583 closed this Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.