From 36c806172127a1a49c87562f80bddf398c73ce5b Mon Sep 17 00:00:00 2001 From: Abhishek Kumar Date: Wed, 5 Mar 2025 13:23:38 +0530 Subject: [PATCH] test: fix failure in test_06_purge_expunged_vm_background_task 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 --- 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 0fe55991059c..4a5da9d1c3ac 100644 --- a/test/integration/smoke/test_purge_expunged_vms.py +++ b/test/integration/smoke/test_purge_expunged_vms.py @@ -259,8 +259,8 @@ def getManagementServerIps(self): 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):