cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simon Waterhouse" <simon.waterho...@citrix.com>
Subject Review Request: Fix for CLOUDSTACK-284
Date Tue, 13 Nov 2012 12:16:52 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8037/
-----------------------------------------------------------

Review request for cloudstack.


Description
-------

Fixed CLOUDSTACK-284 by removing the code that filtered out "Destroyed" VMs if the zone was
specified.


This addresses bug CLOUDSTACK-284.


Diffs
-----

  server/src/com/cloud/vm/UserVmManagerImpl.java a2a548f 

Diff: https://reviews.apache.org/r/8037/diff/


Testing
-------

Ran HTTP "listVirtualMachines" query both with and without "zone" parameter. Test cases, all
performed as expected
1. 0 VMs 
2. 1 running VM
3. 1 running, 1 destroyed VM
4. 2 destroyed VMs


Thanks,

Simon Waterhouse


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message