cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Saksham Srivastava" <saksham.srivast...@citrix.com>
Subject Review Request 13911: CLOUDSTACK-4572: findHostsForMigration API does not return correct host list
Date Fri, 30 Aug 2013 16:38:20 GMT

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

Review request for cloudstack and Devdeep Singh.


Bugs: CLOUDSTACK-4572


Repository: cloudstack-git


Description
-------

Create multi cluster setup. 
Tag host in one cluster with host tag t1. 
Create a service offering using the host tag t1 
Deploy a vm using the tagged service offering. 
Even if tagged/untagged hosts are available across different clusters the api does not list
correct hosts for migration for the deployed vm. 

Expected behavior: 
The api should return the list of suitable/unsuitable hosts

Added fix that creates a deep copy of the the variable allHosts and prevents faulty host list
return.


Diffs
-----

  server/src/com/cloud/server/ManagementServerImpl.java 3341c05 

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


Testing
-------

UI/API displays the correct host list now.


Thanks,

Saksham Srivastava


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