----------------------------------------------------------- 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