GeezerVison

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

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

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

@Runseb @DaanHoogland @remibergsma

Does this one needs 2 LGTM as well?

I'm willing to close a few PRs.

Cheers,
Wilder

changes looks good to me.

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

LGTM

@DaanHoogland

I already merged :)

Got a LGTM from @karuturi

Thanks, guys!