detach cdrom if host installed

app/models/concerns/foreman_xen/host_extensions.rb

@@ -3,7 +3,10 @@ module HostExtensions
     extend ActiveSupport::Concern
 
     def built(installed = true)
-      compute_resource.cleanup_configdrive(uuid) if compute_resource && compute_resource.type == 'ForemanXen::Xenserver'
+      if compute_resource && compute_resource.type == 'ForemanXen::Xenserver'
+        compute_resource.detach_cdrom(uuid)
+        compute_resource.cleanup_configdrive(uuid)
+      end
       super(installed)
     end
 

app/models/foreman_xen/xenserver.rb

@@ -39,6 +39,15 @@ def iso_library_mountpoint=(path)
       attrs[:iso_library_mountpoint] = mountpoint
     end
 
+    def detach_cdrom(uuid)
+      begin
+        vm = find_vm_by_uuid(uuid)
+        detach_cdrom_vbd(vm) if vm
+      rescue => e
+        logger.error "unable to detach cdrom:#{e}"
+      end
+    end
+
     def cleanup_configdrive(uuid)
       iso_file_name = "foreman-configdrive-#{uuid}.iso"
       begin
@@ -497,6 +506,13 @@ def attach_iso(vm, iso_vdi)
       true
     end
 
+    def detach_cdrom_vbd(vm)
+      cd_drive = client.vbds.find { |v| v.vm == vm && v.type == 'CD' }
+      unless cd_drive&.empty
+        client.eject_vbd cd_drive.reference
+      end
+    end
+
     def find_free_userdevice(vm)
       # Find next free userdevice id for vbd
       # vm.vbds is not current, vm.reload not working.