cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gaurav Aradhye" <gaurav.arad...@clogeny.com>
Subject Re: Review Request 21858: CLOUDSTACK-6758: [Marvin] Detect failed job and throw exception
Date Fri, 23 May 2014 12:31:21 GMT

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

(Updated May 23, 2014, 6:01 p.m.)


Review request for cloudstack and Santhosh Edukulla.


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


Repository: cloudstack-git


Description (updated)
-------

Whenever API returns error message for failed job (not exception), in this case it is returned
to test cases as it is. Instead it should throw exceptions for failed jobs.

With this change, reverted changes done in the test cases (Checking for ERROR_CODE_530) with
commit https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=8eca37056f5bb1eee10f0bedee785142d7c3437a


Diffs
-----

  test/integration/component/test_accounts.py 9b9ae75 
  test/integration/component/test_add_remove_network.py 969d94c 
  test/integration/component/test_affinity_groups.py d31746a 
  test/integration/component/test_assign_vm.py 0b07e9f 
  test/integration/component/test_ip_reservation.py eb193d1 
  test/integration/component/test_non_contiguous_vlan.py 1f74252 
  test/integration/component/test_projects.py cf6cec8 
  test/integration/component/test_volumes.py 4a91e5d 
  test/integration/component/test_vpc.py 31b421c 
  test/integration/component/test_vpc_network.py 8d12ec6 
  test/integration/component/test_vpc_network_lbrules.py 47bdce5 
  test/integration/component/test_vpc_vms_deployment.py 19aa448 
  test/integration/smoke/test_network.py 7cd3973 
  test/integration/smoke/test_nic.py a76e858 
  test/integration/smoke/test_primary_storage.py 5b24b64 
  test/integration/smoke/test_volumes.py cc76e49 
  tools/marvin/marvin/cloudstackConnection.py 8044da7 
  tools/marvin/marvin/codes.py c72a6bd 

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


Testing
-------

Tested all the changed test cases on VMware.


Thanks,

Gaurav Aradhye


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