From b16542c8fc0477dc25df15bb3f8f64677c73fa47 Mon Sep 17 00:00:00 2001 From: "Joshua M. Boniface" Date: Fri, 1 Nov 2024 17:18:19 -0400 Subject: [PATCH] Fix double-appending domain bug Since storage_hosts now includes the storage domain as FQDNs, don't re-append it within vmbuilder. --- daemon-common/vmbuilder.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/daemon-common/vmbuilder.py b/daemon-common/vmbuilder.py index 9a7ff5b7..c1e1bbcf 100644 --- a/daemon-common/vmbuilder.py +++ b/daemon-common/vmbuilder.py @@ -336,11 +336,7 @@ def worker_create_vm( retcode, stdout, stderr = pvc_common.run_os_command("uname -m") vm_data["system_architecture"] = stdout.strip() - monitor_list = list() - monitor_names = config["storage_hosts"] - for monitor in monitor_names: - monitor_list.append("{}.{}".format(monitor, config["storage_domain"])) - vm_data["ceph_monitor_list"] = monitor_list + vm_data["ceph_monitor_list"] = config["storage_hosts"] vm_data["ceph_monitor_port"] = config["ceph_monitor_port"] vm_data["ceph_monitor_secret"] = config["ceph_secret_uuid"]