Skip to content

Conversation

@itamarst
Copy link
Contributor

We're running out of disk space, so make sure we clean up as much as possible.

@exarkun
Copy link
Contributor

exarkun commented Nov 30, 2015

How much does this actually clean up? "Files owned by root" seems like a strange criterion to apply for cleanup. How about all the non-root owned files still taking up space? Presumably they'll eventually lead to an out-of-space condition too.

So maybe this it a good change, but it's hard to evaluate without knowing how much space it's going to save. But it's probably not sufficient to resolve the issue.

@exarkun
Copy link
Contributor

exarkun commented Nov 30, 2015

But it's probably not sufficient to resolve the issue.

Therefore, clicking "changes requested".

@Azulinho
Copy link
Contributor

build.yaml Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please add a comment explaining why we don't want to clean the repo here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have no idea, it was there when I got here.

@jml
Copy link
Contributor

jml commented Dec 1, 2015

Failures look intermittent + FLOC-3564.

Please add the comment & merge when ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants