Skip to content

Commit 43f31dc

Browse files
author
Prasanna Santhanam
committed
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>
1 parent 8f380d9 commit 43f31dc

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

test/integration/smoke/test_deploy_vms_with_varied_deploymentplanners.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -181,11 +181,9 @@ def test_deployvm_userdispersing(self):
181181
)
182182
vm1clusterid = filter(lambda c: c.id == vm1.hostid, self.hosts)[0].clusterid
183183
vm2clusterid = filter(lambda c: c.id == vm2.hostid, self.hosts)[0].clusterid
184-
self.assertNotEqual(
185-
vm1clusterid,
186-
vm2clusterid,
187-
msg="VMs (%s, %s) meant to be dispersed are deployed in the same cluster %s" % (vm1.id, vm2.id, vm1clusterid)
188-
)
184+
if vm1clusterid == vm2clusterid:
185+
self.debug("VMs (%s, %s) meant to be dispersed are deployed in the same cluster %s" % (
186+
vm1.id, vm2.id, vm1clusterid))
189187

190188
@attr(tags=["simulator", "advanced", "basic", "sg"])
191189
def test_deployvm_userconcentrated(self):

0 commit comments

Comments
 (0)