Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

bigger cache size for sccache #2280

Merged
merged 1 commit into from Apr 15, 2019
Merged

bigger cache size for sccache #2280

merged 1 commit into from Apr 15, 2019

Conversation

TriplEight
Copy link
Contributor

No description provided.

@TriplEight TriplEight added A3-in_progress Pull request is in progress. No review needed at this stage. M1-ci labels Apr 15, 2019
@parity-cla-bot
Copy link

It looks like @TriplEight signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

1 similar comment
@parity-cla-bot
Copy link

It looks like @TriplEight signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@TriplEight TriplEight changed the title WIP: bigger cache size for sccache bigger cache size for sccache Apr 15, 2019
@TriplEight TriplEight removed the A3-in_progress Pull request is in progress. No review needed at this stage. label Apr 15, 2019
gabreal
gabreal approved these changes Apr 15, 2019
View reviewed changes
Copy link
Member

@gabreal gabreal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just from curiosity: what happens if the cache size is exceeded? can we monitor this?

@TriplEight
Copy link
Contributor Author

TriplEight commented Apr 15, 2019 •
edited

just from curiosity: what happens if the cache size is exceeded? can we monitor this?

this can happen:
mozilla/sccache#407

Some of the debugging and logging stuff in Parity-Ethereum's CI is set up to monitor sccache's behavior.

@gabreal gabreal merged commit 22ea827 into master Apr 15, 2019
@gabreal gabreal deleted the sccache_more_storage branch April 15, 2019 16:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@General-Beck General-Beck General-Beck approved these changes

@gabreal gabreal gabreal approved these changes

@bkchr bkchr bkchr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@TriplEight @parity-cla-bot @General-Beck @gabreal @bkchr