DaanHoogland commented on code in PR #8508: URL: https://github.com/apache/cloudstack/pull/8508#discussion_r1629008188
########## test/integration/smoke/test_cluster_drs.py: ########## @@ -256,7 +294,8 @@ def test_02_balanced_drs_algorithm(self): migration["virtualmachineid"]: migration["destinationhostid"] for migration in migrations } - self.assertEqual(len(vm_to_dest_host_map), 1, msg="DRS plan should have 1 migrations") + # this is one if no svm is considered to be migrated, it might be higher + self.assertTrue(len(vm_to_dest_host_map) <= 1, msg="DRS plan should have 1 migrations") Review Comment: ```suggestion self.assertTrue(len(vm_to_dest_host_map) >= 1, msg="DRS plan should have at least 1 migrations") ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org