FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Cleanup stale files in /tmp which is filling up during some runs by rsafonseca · Pull Request #531 · apache/cloudstack · GitHub

Repository navigation

Cleanup stale files in /tmp which is filling up during some runs - #531

Merged
asfgit merged 2 commits into
apache:masterfrom
rsafonseca:fixtravis3
Jul 1, 2015
Merged

asfgit merged 2 commits into
apache:masterfrom
rsafonseca:fixtravis3

Conversation

Copy link
Copy Markdown
Contributor

Output status of disk and memory on build host for some extra info on build environment before start

This should fix the travis problem made visible in this run:
https://travis-ci.org/apache/cloudstack/jobs/68291716

asfbot commented Jun 25, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #660 SUCCESS
This pull request looks good

asfbot commented Jun 25, 2015

Copy link
Copy Markdown

cloudstack-pull-requests #661 SUCCESS
This pull request looks good

Copy link
Copy Markdown
Contributor

LGTM

I executed the command on one of my machines and it's fine - although my eyes could already see it was.

Result:

[wrodrigues@mct-wrodrigues ~]$ sudo find /tmp -type f -mtime +2 | grep -v "sudo lsof | grep /tmp |awk '{print $9}'|sed -e '1 d' |sort |uniq | tr \\n \|"
/tmp/libvirt/pool_images.xml
/tmp/libvirt/pool_iso.xml

Remark: I removed the "| xargs sudo rm -vf" part because I did not want to remove my files

Cheers,
Wilder

Copy link
Copy Markdown
Contributor

@Runseb @DaanHoogland @remibergsma

Does this one needs 2 LGTM as well?

I'm willing to close a few PRs.

Cheers,
Wilder

karuturi commented Jul 1, 2015

Copy link
Copy Markdown
Member

changes looks good to me.

asfgit merged commit e8c0c8f into apache:master Jul 1, 2015
asfgit pushed a commit that referenced this pull request Jul 1, 2015

Copy link
Copy Markdown
Contributor

@wilderrodrigues let's stick to that for now (I kind of did the last few findbugs thing to see what happened with them:)

LGTM

Copy link
Copy Markdown
Contributor

@DaanHoogland

I already merged :)

Got a LGTM from @karuturi

Thanks, guys!

yadvr pushed a commit that referenced this pull request Jan 20, 2021
Missing internationalization on the search bar and pagination "Showing X results" and pagination dropdown on various views.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

6 participants


Back | FazBrowse Home | New Git URL