Skip to content

Conversation

@rsafonseca
Copy link
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
Copy link

asfbot commented Jun 25, 2015

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

@asfbot
Copy link

asfbot commented Jun 25, 2015

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

@wilderrodrigues
Copy link
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

@wilderrodrigues
Copy link
Contributor

@Runseb @DaanHoogland @remibergsma

Does this one needs 2 LGTM as well?

I'm willing to close a few PRs.

Cheers,
Wilder

@karuturi
Copy link
Member

karuturi commented Jul 1, 2015

changes looks good to me.

@asfgit asfgit merged commit e8c0c8f into apache:master Jul 1, 2015
asfgit pushed a commit that referenced this pull request Jul 1, 2015
@DaanHoogland
Copy link
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

@wilderrodrigues
Copy link
Contributor

@DaanHoogland

I already merged :)

Got a LGTM from @karuturi

Thanks, guys!

rohityadavcloud 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 <[email protected]>
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