Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/894
---
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
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/894#issuecomment-151394435
code LGTM. Its a nice improvement to have. Since remi already ran the
tests, I am merging this.
---
If your project is set up for it, you can reply to this email
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/894#issuecomment-151130113
LGTM, based on a set of tests that I run on this branch:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardwa
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/894#issuecomment-150948834
@sudhansu7 Can you please rebase with current master so I can run some
tests? Thanks!
---
If your project is set up for it, you can reply to this email and have
GitHub user sudhansu7 opened a pull request:
https://github.com/apache/cloudstack/pull/894
CLOUDSTACK-8917 : Instance tab takes long time to load with 12K Vms
modified sql that is used for retrieving vm count .
In load test environment listVirtualmachine takes 8-11 sec to lo
