---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11045/
---
Review request for cloudstack and Prachi Damle.
Description
---
Changes fo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11045/#review20520
---
Thanks for a clean patch and an extensive unit test. Have following
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11045/
---
(Updated May 15, 2013, 2:08 p.m.)
Review request for cloudstack and Prachi Daml
> On May 14, 2013, 12:16 a.m., Prachi Damle wrote:
> > Thanks for a clean patch and an extensive unit test. Have following review
> > comments:
> >
> > 1) When we list the VMs on a host (List userVms =
> > _vmDao.listByHostId(host);), also consider the VMs that are stopped on this
> > host (l
- Devdeep
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11045/#review20520
---
On May 15, 2013, 2:08 p.m., Devdeep Singh wrote:
>
> -
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11045/#review20619
---
Ship it!
The changes in the patch look good. Please also add an int
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11045/
---
(Updated May 16, 2013, 1:14 p.m.)
Review request for cloudstack and Prachi Daml
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11045/
---
(Updated May 16, 2013, 1:15 p.m.)
Review request for cloudstack and Prachi Daml