----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21554/#review49093 -----------------------------------------------------------
Commit a40f2315fdc761309107cc5c88fbd44932edf7e3 in cloudstack's branch refs/heads/4.4 from Gaurav Aradhye [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a40f231 ] CLOUDSTACK-6657: Adding condition for checking error code 530 across test cases Signed-off-by: SrikanteswaraRao Talluri <tall...@apache.org> (cherry picked from commit 8eca37056f5bb1eee10f0bedee785142d7c3437a) Conflicts: test/integration/smoke/test_network.py - ASF Subversion and Git Services On May 16, 2014, 2:03 p.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21554/ > ----------------------------------------------------------- > > (Updated May 16, 2014, 2:03 p.m.) > > > Review request for cloudstack and Santhosh Edukulla. > > > Bugs: CLOUDSTACK-6657 > https://issues.apache.org/jira/browse/CLOUDSTACK-6657 > > > Repository: cloudstack-git > > > Description > ------- > > Adding explicit check for error code 530 in the response received form the > server wherever applicable. > I have made these changes in all those files which failed due to having wrong > assert (Build #348 KVM regression). > > Will add more of these changes in other files whenever found failing due to > above reason in the build logs. > > > Diffs > ----- > > test/integration/component/test_accounts.py cf5f2b6 > test/integration/component/test_add_remove_network.py c17293d > test/integration/component/test_affinity_groups.py d0fa2f7 > test/integration/component/test_assign_vm.py 24b63a6 > test/integration/component/test_ip_reservation.py c84f63c > test/integration/component/test_non_contiguous_vlan.py 4609af9 > test/integration/component/test_projects.py c593fb6 > test/integration/component/test_volumes.py b5b08e2 > test/integration/component/test_vpc.py e9d396e8 > test/integration/component/test_vpc_network.py a449948 > test/integration/component/test_vpc_network_lbrules.py a5625e7 > test/integration/component/test_vpc_vms_deployment.py 1c1f93d > test/integration/smoke/test_network.py 6beb071 > test/integration/smoke/test_nic.py 592a839 > test/integration/smoke/test_primary_storage.py 3147d86 > test/integration/smoke/test_volumes.py 5908e2b > > Diff: https://reviews.apache.org/r/21554/diff/ > > > Testing > ------- > > Tested all test suites with python command. > Tested only few test suites by running them, not all. Same working change has > been applied to all files. > > > File Attachments > ---------------- > > Rebased patch against master > > https://reviews.apache.org/media/uploaded/files/2014/05/16/23566db1-7f6c-46bf-9444-878d9a6e7630__0001-CLOUDSTACK-6657-Adding-condition-for-checking-error-.patch > > > Thanks, > > Gaurav Aradhye > >