This is an automated email from the ASF dual-hosted git repository.
shwstppr pushed a commit to branch 4.20
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/4.20 by this push:
new 35a7438033a test: fix failure in
test_06_purge_expunged_vm_background_task (#10501)
35a7438033a is described below
commit 35a7438033a6456b7e26c0722d4c66f325eaab90
Author: Abhishek Kumar <abhishek.mr...@gmail.com>
AuthorDate: Thu Mar 6 09:25:22 2025 +0530
test: fix failure in test_06_purge_expunged_vm_background_task (#10501)
Failures seen in
https://github.com/apache/cloudstack/pull/10006#issuecomment-2691067265 and
others.
With https://github.com/apache/cloudstack/pull/9773, the response of
listManagementServers API has been changed. serviceip has been renamed
to ipaddress. This was causing not all MS getting restarted and purge
b/g task not being able to run. The code handling API response in the
test has been updated.
Signed-off-by: Abhishek Kumar <abhishek.mr...@gmail.com>
---
test/integration/smoke/test_purge_expunged_vms.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/test/integration/smoke/test_purge_expunged_vms.py
b/test/integration/smoke/test_purge_expunged_vms.py
index 0fe55991059..4a5da9d1c3a 100644
--- a/test/integration/smoke/test_purge_expunged_vms.py
+++ b/test/integration/smoke/test_purge_expunged_vms.py
@@ -259,8 +259,8 @@ class TestPurgeExpungedVms(cloudstackTestCase):
active_server_ips = []
active_server_ips.append(self.mgtSvrDetails["mgtSvrIp"])
for idx, server in enumerate(servers):
- if server.state == 'Up' and server.serviceip !=
self.mgtSvrDetails["mgtSvrIp"]:
- active_server_ips.append(server.serviceip)
+ if server.state == 'Up' and server.ipaddress !=
self.mgtSvrDetails["mgtSvrIp"]:
+ active_server_ips.append(server.ipaddress)
return active_server_ips
def restartAllManagementServers(self):
