You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by rsafonseca <gi...@git.apache.org> on 2015/06/26 00:59:10 UTC

[GitHub] cloudstack pull request: Cleanup stale files in /tmp which is fill...

GitHub user rsafonseca opened a pull request:

    https://github.com/apache/cloudstack/pull/531

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

    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

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/rsafonseca/cloudstack fixtravis3

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/531.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #531
    
----
commit c980003c37602e2402225bddea9edcc8539a6e11
Author: Rafael da Fonseca <rs...@gmail.com>
Date:   2015-06-25T21:12:17Z

    Cleanup stale files in /tmp which is filling up during some runs
    Output status of disk and memory on build host

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Cleanup stale files in /tmp which is fill...

Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:

    https://github.com/apache/cloudstack/pull/531#issuecomment-117517482
  
    @DaanHoogland 
    
    I already merged :)
    
    Got a LGTM from @karuturi 
    
    Thanks, guys!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Cleanup stale files in /tmp which is fill...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/531


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Cleanup stale files in /tmp which is fill...

Posted by DaanHoogland <gi...@git.apache.org>.
Github user DaanHoogland commented on the pull request:

    https://github.com/apache/cloudstack/pull/531#issuecomment-117517228
  
    @wilderrodrigues let's stick to that for now (I kind of did the last few findbugs thing to see what happened with them:)
    
    LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Cleanup stale files in /tmp which is fill...

Posted by karuturi <gi...@git.apache.org>.
Github user karuturi commented on the pull request:

    https://github.com/apache/cloudstack/pull/531#issuecomment-117443894
  
    changes looks good to me. 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Cleanup stale files in /tmp which is fill...

Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:

    https://github.com/apache/cloudstack/pull/531#issuecomment-117441816
  
    @runseb @DaanHoogland @remibergsma 
    
    Does this one needs 2 LGTM as well?
    
    I'm willing to close a few PRs.
    
    Cheers,
    Wilder


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request: Cleanup stale files in /tmp which is fill...

Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:

    https://github.com/apache/cloudstack/pull/531#issuecomment-117441768
  
    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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---