Skip to content

Commit

Permalink
Fix double-appending domain bug
Browse files Browse the repository at this point in the history
Since storage_hosts now includes the storage domain as FQDNs, don't
re-append it within vmbuilder.
  • Loading branch information
joshuaboniface committed Nov 1, 2024
1 parent de0c7e3 commit b16542c
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions daemon-common/vmbuilder.py
Original file line number Diff line number Diff line change
Expand Up @@ -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"]

Expand Down

0 comments on commit b16542c

Please sign in to comment.