Liron Ar has uploaded a new change for review.

Change subject: core : update of bootable disk
......................................................................

core : update of bootable disk

when updating a bootable disk, a CDA message was always shown because of
a check that the vms that the disk is attached to doesn't contain boot
disk - this disk shouldn't be done if the disk was already bootable
before the update operation.

Change-Id: Ibc0e96274e408306d2444f8fa01a45daf38724f7
Signed-off-by: Liron Aravot <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/18/15218/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java
index cc4df0f..dd45b2e 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java
@@ -130,7 +130,7 @@
         }
 
         // Validate update boot disk.
-        if (newDisk.isBoot()) {
+        if (newDisk.isBoot() && !oldDisk.isBoot()) {
             if (!validate(noVmsContainBootableDisks(vmsDiskPluggedTo))) {
                 return false;
             }


--
To view, visit http://gerrit.ovirt.org/15218
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibc0e96274e408306d2444f8fa01a45daf38724f7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Ar <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to