svenvogel commented on a change in pull request #3524: Fix VR 
bootstrapping/connection state in KVM
URL: https://github.com/apache/cloudstack/pull/3524#discussion_r307642511
 
 

 ##########
 File path: systemvm/debian/opt/cloud/bin/setup/bootstrap.sh
 ##########
 @@ -85,12 +85,18 @@ config_guest() {
           sed -i "s/%/ /g" $CMDLINE
           ;;
      kvm)
-          # Configure hot-plug
-          modprobe acpiphp || true
-          modprobe pci_hotplug || true
+          # Configure kvm hotplug support
+          if grep -E 'CONFIG_HOTPLUG_PCI=y|CONFIG_HOTPLUG_PCI_ACPI=y' 
/boot/config-`uname -r`; then
+            log_it "acpiphp and pci_hotplug module already compiled in"
+          else
+            modprobe acpiphp 2> /dev/null && log_it "acpiphp module loaded" || 
true
+            modprobe pci_hotplug 2> /dev/null && log_it "pci_hotplug module 
loaded" || true
+          fi
+
           sed -i -e "/^s0:2345:respawn.*/d" /etc/inittab
           sed -i -e "/6:23:respawn/a\s0:2345:respawn:/sbin/getty -L 115200 
ttyS0 vt102" /etc/inittab
-          systemctl enable --now qemu-guest-agent
 
 Review comment:
   @rhtyd yes normally it should but qemu-guest-agent it a sysvinit and 
unfortunately with this redirect there is no startup. 
   
   ```sh
   [root@kvm08 /]# /lib/systemd/systemd-sysv-install --help
   systemd-sysv-install version 1.7.4 - Copyright (C) 1997-2000 Red Hat, Inc.
   This may be freely redistributed under the terms of the GNU Public License.
   
   usage:   systemd-sysv-install <enable|disable|is-enabled> [name]
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to