cloudstack/plugins
Rajani Karuturi d9bd01266f Merge pull request #1829 from syed/hvm-volume-attach-restart-fix
CLOUDSTACK-9363: Fix HVM VM restart bug in XenServerHere is the longer description of the problem:

By default XenServer limits HVM guests to only 4 disks. Two of those are reserved for the ROOT disk (deviceId=0) and CD ROM (device ID=3) which means that we can only attach 2 data disks. This limit however is removed when Xentools is installed on the guest. The information that a guest has Xentools installed and can handle more than 4 disks is stored in the VM metadata on XenServer. When a VM is shut down, Cloudstack removes the VM and all the metadata associated with the VM from XenServer. Now, when you start the VM again, even if it has Xentools installed, it will default to only 4 attachable disks.

Now this problem manifests itself when you have a HVM VM and you stop and start it with more than 2 data disks attached. The VM fails to start and the only way to start the VM is to detach the extra disks and then reattach them after the VM start.

In this fix, I am removing the check which is done before creating a `VBD` which enforces this limit. This will not affect current workflow and will fix the HVM issue.

@koushik-das this is related to the "autodetect" feature that you introduced a while back (https://issues.apache.org/jira/browse/CLOUDSTACK-8826). I would love your review on this fix.

* pr/1829:
  Fix HVM VM restart bug in XenServer

Signed-off-by: Rajani Karuturi <rajani.karuturi@accelerite.com>
2017-02-28 05:42:59 +05:30
..
acl Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
affinity-group-processors Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
alert-handlers Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
api Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
database Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
dedicated-resources Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
deployment-planners Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
event-bus Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
file-systems/netapp Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
ha-planners/skip-heurestics Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
host-allocators/random Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
hypervisors Merge pull request #1829 from syed/hvm-volume-attach-restart-fix 2017-02-28 05:42:59 +05:30
network-elements Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
outofbandmanagement-drivers/ipmitool Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
storage Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
storage-allocators/random Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
user-authenticators Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30
pom.xml Updating pom.xml version numbers for release 4.9.3.0-SNAPSHOT 2017-01-06 10:40:15 +05:30