You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ts...@apache.org on 2013/07/02 15:52:51 UTC
[4/4] git commit: updated refs/heads/4.2 to e950b3a
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 <ts...@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 <ts...@apache.org>
Authored: Tue Jul 2 19:12:53 2013 +0530
Committer: Prasanna Santhanam <ts...@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):