From 43f31dcc660920da231b2eaf06eb4c599fd6a366 Mon Sep 17 00:00:00 2001 From: Prasanna Santhanam Date: Tue, 2 Jul 2013 19:12:53 +0530 Subject: [PATCH] Only print debug message when the planner fails to disperse User Dispersion is a best effort of deployment planning and cannot always guarantee the dispersion logic will work. However this does not fail the deployment. Adjust the test so as to print a debug and exit instead of fail. Signed-off-by: Prasanna Santhanam --- .../test_deploy_vms_with_varied_deploymentplanners.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/test/integration/smoke/test_deploy_vms_with_varied_deploymentplanners.py b/test/integration/smoke/test_deploy_vms_with_varied_deploymentplanners.py index ab44a2be083..a08537e124f 100644 --- a/test/integration/smoke/test_deploy_vms_with_varied_deploymentplanners.py +++ b/test/integration/smoke/test_deploy_vms_with_varied_deploymentplanners.py @@ -181,11 +181,9 @@ class TestDeployVmWithVariedPlanners(cloudstackTestCase): ) vm1clusterid = filter(lambda c: c.id == vm1.hostid, self.hosts)[0].clusterid vm2clusterid = filter(lambda c: c.id == vm2.hostid, self.hosts)[0].clusterid - self.assertNotEqual( - vm1clusterid, - vm2clusterid, - msg="VMs (%s, %s) meant to be dispersed are deployed in the same cluster %s" % (vm1.id, vm2.id, vm1clusterid) - ) + if vm1clusterid == vm2clusterid: + self.debug("VMs (%s, %s) meant to be dispersed are deployed in the same cluster %s" % ( + vm1.id, vm2.id, vm1clusterid)) @attr(tags=["simulator", "advanced", "basic", "sg"]) def test_deployvm_userconcentrated(self):