cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Santhosh Edukulla" <santhosh.eduku...@citrix.com>
Subject Review Request 17995: Added fix for CLOUDSTACK - 6164
Date Mon, 24 Feb 2014 15:06:11 GMT

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

Review request for cloudstack and Girish Shilamkar.


Bugs: CLOUDSTACK-6164
    https://issues.apache.org/jira/browse/CLOUDSTACK-6164


Repository: cloudstack-git


Description
-------

Currently,Host class has a create method, uses addHost Command, but 

1. It does not verify the output of addHost command is valid output in all forms. 
2. Tests some times directly goes to using Host post Host.create, but we are not sure whether
the host is up or not, they have to explicitly call again listHosts api call, so every where
host is added list host is called. Instead, the create method now returns the FAILED or host
object based upon the api success and whether the host is up. 
3. Added a try\except block and return FAILED to tests in case of an exception. 


Diffs
-----

  tools/marvin/marvin/config/test_data.py PRE-CREATION 
  tools/marvin/marvin/deployDataCenter.py c8feaaf 

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


Testing
-------

Tested indentation and import changes.


Thanks,

Santhosh Edukulla


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