Change in ovirt-engine[ovirt-engine-4.0]: engine: Clear IDE disk address on bootable change

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: engine: Clear IDE disk address on bootable change
......................................................................


engine: Clear IDE disk address on bootable change

In the following scenario, the VM will fail to start:
1. Create new VM with IDE disk, verify it starts
2. Add second IDE disk
3. Turn off bootable flag from 1st disk, make 2nd disk bootable
4. Start the VM

The root cause of this issue is that we save the device address
of the disk, but we changed the order of the disks.
This will cause a mismatch between the address and the name of the disk.

In order to fix this issue, the address will be cleared for IDE disks
if the 'bootable' attribute changed.

Change-Id: I46847ae9f0870e454e81722834535861b87abed8
Bug-Url: https://bugzilla.redhat.com/1186817
Signed-off-by: Fred Rolland <frolland at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/UpdateVmDiskCommand.java
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, but someone else must approve
  Freddy Rolland: Verified



-- 
To view, visit https://gerrit.ovirt.org/64101
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I46847ae9f0870e454e81722834535861b87abed8
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>