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

(Updated Aug. 5, 2014, 3:32 p.m.)


Review request for cloudstack and Santhosh Edukulla.


Summary (updated)
-----------------

CLOUDSTACK-7247: Fixed Marvin Issue: Hypervisor value was hardcoded to 
XenServer while deploying VM if not specified


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


Repository: cloudstack-git


Description
-------

Hypervisor value was passed as XenServer by default. Changed this to None as 
this is not a compulsory parameter. But this caused failures when the parameter 
value was not passed to the function.


Diffs
-----

  tools/marvin/marvin/lib/base.py eb05a18 

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


Testing
-------

Yes.

@summary: Test to Assign and Remove Load Balancer Rule to an Instance ... === 
TestName: test_07_assign_remove_lbrule_toinstance | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 1 test in 423.142s

OK


Thanks,

Gaurav Aradhye

Reply via email to