cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t..@apache.org
Subject [4/4] git commit: updated refs/heads/4.2 to e950b3a
Date Tue, 02 Jul 2013 13:52:51 GMT
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 <tsp@apache.org>
(cherry picked from commit 43f31dcc660920da231b2eaf06eb4c599fd6a366)


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/e950b3a9
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/e950b3a9
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/e950b3a9

Branch: refs/heads/4.2
Commit: e950b3a9fa6a3c35b7bfe55636016f234ecfa073
Parents: 6e399cc
Author: Prasanna Santhanam <tsp@apache.org>
Authored: Tue Jul 2 19:12:53 2013 +0530
Committer: Prasanna Santhanam <tsp@apache.org>
Committed: Tue Jul 2 19:21:15 2013 +0530

----------------------------------------------------------------------
 .../smoke/test_deploy_vms_with_varied_deploymentplanners.py  | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/e950b3a9/test/integration/smoke/test_deploy_vms_with_varied_deploymentplanners.py
----------------------------------------------------------------------
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 ab44a2b..a08537e 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):


Mime
View raw message